-
Notifications
You must be signed in to change notification settings - Fork 114
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use specific prettier version #1117
Use specific prettier version #1117
Conversation
Homebrew installs packages differently depending on chip architecture -- updated the setup script to reflect this difference.
📟 🛠️ Update `setup_ios_native.sh` to support Apple Silicone
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## service_rewrite_2023 #1117 +/- ##
=====================================================
Coverage 58.80% 58.80%
=====================================================
Files 26 26
Lines 1420 1420
Branches 320 320
=====================================================
Hits 835 835
Misses 585 585
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
If we ever decide to update the Prettier version, we will change it both here and in package.serve.json
.
@@ -7,5 +7,6 @@ jobs: | |||
runs-on: ubuntu-latest | |||
steps: | |||
- uses: actions/checkout@v2 | |||
- run: npx prettier --check www | |||
- run: npx prettier@3.0.3 --check www |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to know we can set it to use a specific version this way!
In 4f173ba I bumped the prettier version to 3.1. Whatever version we are using for the project should be the same as the workflow runs with (as we established in e-mission#1117)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
In 4f173ba I bumped the prettier version to 3.1. Whatever version we are using for the project should be the same as the workflow runs with (as we established in e-mission#1117)
Background
Prettier
failed on the UI Migration Wrap-up PR, and we discovered that it uses the latest Prettier version on GitHub Actions but version 3.0.3 in the project. Therefore, we have decided to stick with the specific version (3.0.3) to prevent this issue from occurring again.