-
Notifications
You must be signed in to change notification settings - Fork 60
style(lint): replace pylint with ruff #675
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
Conversation
3f99d04 to
dab8d9c
Compare
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, thanks!
|
Moving back to draft so I can:
|
2b5712f to
227f667
Compare
This was done with `ruff check --fix --unsafe-fixes`
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.
Thanks, this is great ;)
|
@lengau just to confirm: are you adding the requirements*txt updates here? |
|
Just do it here if you can please :)
…On Tue, Aug 27, 2024, 18:08 Alex Lowe ***@***.***> wrote:
@tigarmo <https://github.com/tigarmo> I was planning on doing #678
<#678> once this is merged,
but if you'd prefer I can put the requirements file here.
—
Reply to this email directly, view it on GitHub
<#675 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAO4MQ744VESDO4NXGPZZLTZTTTGDAVCNFSM6AAAAABMUT2EDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJTGU2TAOJXHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
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.
a thousand blessings upon you and your house
Uh oh!
There was an error while loading. Please reload this page.