-
Notifications
You must be signed in to change notification settings - Fork 66
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
Use Rye instead of Poetry #5
Conversation
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.
overall looking good.
- uses: pre-commit/action@v3.0.1 | ||
with: | ||
extra_args: --all-files --verbose | ||
- uses: eifinger/setup-rye@v2 |
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.
Surely simplest to just run pre-commit in CI like other projects.
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.
It was not simple, I'll add pre-commit here later.
* Push * use rye * Sync instead * Add pip * Add pre-commit * Use rye instead of standalone commands * Lint without pre-commit * Install rye via curl * install properly * Add mkdocs packages * move up
No description provided.