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 ruff, python linter written in rust #737

Merged
merged 1 commit into from
Nov 27, 2022
Merged

Conversation

tgross35
Copy link
Contributor

No description provided.

@asottile
Copy link
Member

lint is a poor name for a hook id -- it should be identifiable as your project since it is used as a key for selecting / deselecting and in error messages

@tgross35
Copy link
Contributor Author

tgross35 commented Oct 27, 2022

Talked with the project maintainer, id ruff was added and lint was marked deprecated removed

First version with this new id will be released at 0:00 utc (v0.0.86) when the cron runs (or sooner) version 0.0.86 with hook id ruff is now latest

@charliermarsh
Copy link

@asottile - This got changed from lint to ruff -- any chance we can add to the list?

@asottile
Copy link
Member

yep! should be good to go now -- just needs a rebase. it wasn't ready when I was pinged so I didn't get a chance to look at this

@charliermarsh
Copy link

@asottile - No prob at all. I can't edit this PR, easier for me to just open a new one?

@asottile
Copy link
Member

I'll just rebase this then, one sec

@charliermarsh
Copy link

Rock, thanks @asottile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants