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

Black as a plugin #738

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Black as a plugin #738

wants to merge 14 commits into from

Conversation

akaihola
Copy link
Owner

@akaihola akaihola commented Sep 15, 2024

@akaihola akaihola added the enhancement New feature or request label Sep 15, 2024
@akaihola akaihola added this to the Darker 3.1.0 milestone Sep 15, 2024
@akaihola akaihola self-assigned this Sep 15, 2024
@akaihola akaihola added the many-formatters Support for multiple code formatters and skipping formatting label Sep 15, 2024
@akaihola akaihola force-pushed the black-as-plugin branch 3 times, most recently from b9946be to f9301f3 Compare September 17, 2024 07:36
@akaihola akaihola marked this pull request as draft September 17, 2024 07:39
@akaihola akaihola force-pushed the black-as-plugin branch 3 times, most recently from e450135 to 9d28eca Compare September 24, 2024 06:03
@akaihola akaihola force-pushed the black-as-plugin branch 9 times, most recently from df3e7b8 to 6be3033 Compare September 25, 2024 20:00
@akaihola akaihola force-pushed the black-as-plugin branch 7 times, most recently from b8451c6 to ff047ac Compare October 12, 2024 19:58
@akaihola akaihola force-pushed the black-as-plugin branch 2 times, most recently from 61c026b to d1f4101 Compare October 18, 2024 17:45
@akaihola akaihola marked this pull request as ready for review October 19, 2024 17:30
@akaihola akaihola mentioned this pull request Oct 19, 2024
2 tasks
@akaihola
Copy link
Owner Author

akaihola commented Oct 19, 2024

Hi @clintonsteiner, this is the PR leading up to #744 (ruff support). You already reviewed the draft PR #744 which also contains these commits. I did a bit of clean-up and rebased this one again, and I believe this is now ready to be reviewed and merged. After this one I'd like to review and merge #759 (drop Black dependency) as well.

Oh, and as with #744, I tried to create a logical sequence of commits to make this easier to review commit by commit.

@akaihola akaihola mentioned this pull request Oct 19, 2024
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request many-formatters Support for multiple code formatters and skipping formatting
Projects
Development

Successfully merging this pull request may close these issues.

1 participant