Skip to content
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

Add prettier and eslint workflow #4

Closed
wants to merge 6 commits into from
Closed

Conversation

wjymtg
Copy link
Collaborator

@wjymtg wjymtg commented Dec 6, 2024

新加的workflow会先用prettier format然后运行eslint

  • prettier跑完之后会自动提交,目前没有额外参数应该就是默认设置,具体怎么配还得二位决定下 @Of-qinhai @linjm8780860
  • eslint的设置在eslint.config.js里,我把目前存在的问题都设置成warning了所以不会触发报错
    • css其实也有错误但不知道怎么调成warning,就先跳过了

Copy link

github-actions bot commented Dec 6, 2024

Your code does not meet our eslint standards.
Please run npm install -g eslint @eslint/js typescript-eslint eslint-plugin-vue && npx eslint src to lint your code.

Copy link

github-actions bot commented Dec 6, 2024

Your code does not meet our eslint standards.
Please run npm install -g eslint @eslint/js typescript-eslint eslint-plugin-vue && npx eslint src to lint your code.

@wjymtg wjymtg self-assigned this Dec 6, 2024
@wjymtg wjymtg changed the title test workflow trigger Add prettier and eslint workflow Dec 6, 2024
@wjymtg
Copy link
Collaborator Author

wjymtg commented Dec 6, 2024

@Of-qinhai 本地加了eslint和prettier,我修改一下workflow再合到dev_bbs

@wjymtg wjymtg closed this Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant