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

Remove flake8-to-ruff #9329

Merged
merged 1 commit into from
Dec 31, 2023
Merged

Remove flake8-to-ruff #9329

merged 1 commit into from
Dec 31, 2023

Conversation

charliermarsh
Copy link
Member

Summary

We stopped releasing this a while ago and no longer advertise it anywhere. IMO, we should remove it so that we stop paying the cost of maintaining it. If we want to revive it, we can always do so from Git.

@charliermarsh charliermarsh requested a review from zanieb December 31, 2023 12:19
@charliermarsh charliermarsh marked this pull request as ready for review December 31, 2023 12:19
@charliermarsh charliermarsh added the internal An internal refactor or improvement label Dec 31, 2023
Copy link
Contributor

github-actions bot commented Dec 31, 2023

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@charliermarsh charliermarsh merged commit 772e5d5 into main Dec 31, 2023
40 checks passed
@charliermarsh charliermarsh deleted the charlie/flake8-to-ruff branch December 31, 2023 16:09
ajayk added a commit to wolfi-dev/os that referenced this pull request Jan 2, 2024
flate8-to-ruff was removed  from upstream astral-sh/ruff#9329

Signed-off-by: Ajay Kemparaj <ajaykemparaj@gmail.com>
@akx
Copy link
Contributor

akx commented Jan 5, 2024

Ah, it might be a good idea to push one more (binary-identical?) 0.0.233.1 release to PyPI with a prominent "This project has been archived, see #9329" warning in the readme...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal An internal refactor or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants