-
Couldn't load subscription status.
- Fork 239
[CI] Add ruff check #538
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
base: dev
Are you sure you want to change the base?
[CI] Add ruff check #538
Conversation
|
Before adding this to CI it should probably be added to the documentation. Otherwise many people contributing will not be aware of it and will not format their code properly which will cause CI to fail. It might also make sense to have an easy way for people who don't already have it installed to run ruff locally, maybe using pre-commit. |
|
This is a large PR, changing plugins, riscv-isac source code, adding new things in the installation guide, updating CI. All of these things have to be documented first, otherwise we will not be able to achieve good code formatting and CI will keep failing. |
Agree, I'm working on that, maybe added to CONTRIBUTION.md At the same time, I noticed that because many places in the code use This PR(include ci) does not perform any formatting(may be a few small places that I modified casually); the modifications are just to resolve lint errors or warnings. |
d5a280b to
5662ff3
Compare
|
More commits here and I've allready test it with all cgfs for RV64 you can check current CI result at my fork repo https://github.com/trdthg/riscv-arch-test/actions
|
|
I will break this PR into some smaller parts |
…eval()` working correctly Signed-off-by: MingZhu Yan <[email protected]>
- Add precommit config - Update CONTRIBUTION.md, Add "Prepare" section Signed-off-by: MingZhu Yan <[email protected]>
Description
Currently, this PR includes the following content
Auto fix by ruff: Most of it is about imports
Manually fix
import *with specific content==->isAdd CI, currently only running the ruff check command, I would like to separate
ruff format --checkto the next PR