-
Notifications
You must be signed in to change notification settings - Fork 717
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
chore: add prettier script #18
base: main
Are you sure you want to change the base?
Conversation
chenshuai2144
commented
Oct 26, 2021
•
edited
Loading
edited
- 所有的文件都应该保持一个代码风格
- pr 需要 ci
之前加了 yarn.lock 在.gitignore. 需要同步删下 |
@@ -0,0 +1,37 @@ | |||
name: lint CI | |||
|
|||
on: [push, pull_request] |
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.
建议只在 pull_request
运行
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.
push 也需要的,不然合并到matser的时候master就不会检查了。
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.
那可以限定一下分支,只在 main
test
release
这些重要的分支触发。
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.
没必要的,github 非常 rich,牺牲一点时间保证代码的正确性是非常合算的