-
Notifications
You must be signed in to change notification settings - Fork 24
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
dev: Added package management with Poetry
, Ruff
for code static analysis
#567
Conversation
Poetry
Poetry
Poetry
Poetry
, Ruff
for code static analysis
@fukusuket Thanks for this!
It would be nice to figure out what rules are not being able to get converted. Can you add the filename to the error output? |
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
@YamatoSecurity @hitenkoku
|
@fukusuket Thanks! Looks good now. I updated the English changelog. Could you update the Japanese one for me? |
@YamatoSecurity |
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.
@fukusuket Thank you so much! All looks good, so I will merge this then.
What Changed
Ruff (made with Rust), a Python code static analysis tool, looked good, so I decided to try it out :)
Add the following tools for the development environment.
Cargo
)Clippy
)This PR does not affect script logic. (This is a PR for improving the development environment)
Evidence
Test Environment
Test(README)
Complete the script as follows according to the steps in the README.
Test(Code static analysis)
I was able to execute the
Ruff
command and check the code static analysis results.I would appreciate it if you could review when you have time🙏