Skip to content

chore: remove black#1985

Merged
anakin87 merged 4 commits intomainfrom
rm-black
Jun 24, 2025
Merged

chore: remove black#1985
anakin87 merged 4 commits intomainfrom
rm-black

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Jun 23, 2025

Related Issues

Proposed Changes:

How did you test it?

CI

Checklist

@anakin87 anakin87 marked this pull request as ready for review June 23, 2025 09:07
@anakin87 anakin87 requested a review from a team as a code owner June 23, 2025 09:07
@anakin87 anakin87 requested review from Amnah199 and removed request for a team June 23, 2025 09:07
@anakin87 anakin87 self-assigned this Jun 23, 2025
skip-string-normalization = true

[tool.ruff]
target-version = "py38"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we require python => 3.9, should we upgrade this to py39?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes!
I want to do this in another PR (to be done on Haystack too), since this would re-format all the repository.

Copy link
Copy Markdown
Contributor

@Amnah199 Amnah199 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets Go!

@anakin87 anakin87 merged commit 15e12c7 into main Jun 24, 2025
184 checks passed
@anakin87 anakin87 deleted the rm-black branch June 24, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment