-
Notifications
You must be signed in to change notification settings - Fork 117
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
[ui] Run prettier over the codebase #946
Conversation
thomasdavis
commented
Mar 13, 2021
- Seems harmless, app builds, and tested a few things, everything seems fine.
- Some dist files got run over but there temp and should also be fine.
- I think everyone knows how to switch on prettier
VUE_APP_DEPLOYMENT_TAG: sandpit |
Codecov Report
@@ Coverage Diff @@
## develop #946 +/- ##
===========================================
- Coverage 67.81% 67.63% -0.18%
===========================================
Files 76 76
Lines 3048 3059 +11
Branches 370 378 +8
===========================================
+ Hits 2067 2069 +2
- Misses 876 885 +9
Partials 105 105
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Nice work! Will this fail the build if you commit a file that isn't run over with prettier? |
I second this. Pre-commit hook? |
.github/workflows/prettier.yml
Outdated
# ESLint and Prettier must be in `package.json` | ||
- name: Install Node.js dependencies | ||
run: yarn | ||
- name: Run linters |
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.
Is it a waste to make a whole flow for this? Would it be better if we just add it to yarn build
in the ui?
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.
Also this is only running it on .js files
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.
Yea. We should just run prettier before yarn build
VUE_APP_DEPLOYMENT_TAG: sandpit |
Is the config this is working off in here? I can't seem to find it. Seems like this is adding the "arrowParens" option to the core lib but the config has not been edited? |
VUE_APP_DEPLOYMENT_TAG: sandpit |
VUE_APP_DEPLOYMENT_TAG: sandpit |
VUE_APP_DEPLOYMENT_TAG: sandpit |
VUE_APP_DEPLOYMENT_TAG: mainnet |