-
Notifications
You must be signed in to change notification settings - Fork 24.6k
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 #2292
base: master
Are you sure you want to change the base?
add ruff #2292
Conversation
0l
On Dec 30, 2022 6:23 PM, "Pedro Fonseca" ***@***.***> wrote:
What is this Python project?
This is in python library for linter
What's the difference between this Python project and similar ones?
[image: Bar chart with benchmark results]
<https://user-images.githubusercontent.com/1309177/187504482-6d9df992-a81d-4e86-9f6a-d958741c8182.svg>
- 10-100x faster than existing linters
- Installable via pip
- hammer_and_wrench pyproject.toml support
- Native re-implementations of popular Flake8 plugins, like
flake8-bugbear <https://pypi.org/project/flake8-bugbear/>
- Monorepo-friendly configuration via hierarchical and cascading settings
…--
Anyone who agrees with this pull request could submit an *Approve* review
to it.
------------------------------
You can view, comment on, or merge this pull request online at:
#2292
Commit Summary
- a50a140
<a50a140>
add ruff in code linters
File Changes
(1 file <https://github.com/vinta/awesome-python/pull/2292/files>)
- *M* README.md
<https://github.com/vinta/awesome-python/pull/2292/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5>
(2)
Patch Links:
- https://github.com/vinta/awesome-python/pull/2292.patch
- https://github.com/vinta/awesome-python/pull/2292.diff
—
Reply to this email directly, view it on GitHub
<#2292>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AX6RBONI7ZHDUUEAERK2M6TWP4SCBANCNFSM6AAAAAATNB6F44>
.
You are receiving this because you are subscribed to this thread.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.
Nice one
👍 |
5d5f6f9
to
a50a140
Compare
This should be merge. Great linter. |
Agree with @Kub-AT . This PR should get merged as Ruff is getting more popular |
👍🏻 |
Bumping this issue, Ruff is my favorite and go to linter it should be featured on here surprised it is not |
Agreed! So... did you approve this PR so we can get enough votes to move it through, @Commando-Brando ?
|
@bowmanjd didn't read the whole issue, oops, but it has been approved, cheers! |
Agreed! @Kub-AT could you approve the PR (that is what counts as a vote) |
Yes, indeed! Could you approve the PR, @Owen-CH-Leung, then (approval is what counts as a vote; we need 20 to see ruff make the awesome list) |
more 3 likes 🙏 |
@vinta 20 likes |
Thanks for a quick update, @peidrao! |
BTW, in the meantime even the CPython repository started using ruff: |
@peidrao can you do a rebase and a squash? |
44e6120
to
0012984
Compare
Co-authored-by: Aliaksei Urbanski <aliaksei.urbanski@gmail.com>
I have completed the rebase and squash. |
@vinta ready to merge. |
This comment has been minimized.
This comment has been minimized.
|
|
What is this Python project?
This is in python library for linter
What's the difference between this Python project and similar ones?
--
Anyone who agrees with this pull request could submit an Approve review to it.