Skip to content

Conversation

@harshil21
Copy link
Member

  • Black has reached a new stable release, which brings the new stable style. So it has lots of fixes for us :)

  • Ruff has a new release sorting all our __slots__.

  • flake8 is angry that Black has changed so I silenced it.

  • ruff now runs first in pre-commit as per @clot27's suggestion

@harshil21 harshil21 added the 🛠 code-quality change type: code-quality label Feb 1, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hey! Looks like you edited the (optional) requirements or the pre-commit hooks. I'm just a friendly reminder to keep the additional dependencies for the hooks in sync with the requirements :)

@github-actions github-actions bot removed the 🛠 code-quality change type: code-quality label Feb 1, 2024
@harshil21 harshil21 added the 🛠 code-quality change type: code-quality label Feb 1, 2024
They release so fast we can't even merge one PR
@harshil21 harshil21 added the 📋 pending-merge work status: pending-merge label Feb 5, 2024
@Bibo-Joshi
Copy link
Member

The issues reported by DeepSource appear to be invalid. probably they changed the configuration again …

@Bibo-Joshi Bibo-Joshi merged commit c23eb8e into master Feb 5, 2024
@Bibo-Joshi Bibo-Joshi deleted the bump-ruff branch February 5, 2024 18:24
@harshil21 harshil21 removed the 📋 pending-merge work status: pending-merge label Feb 5, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Feb 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🛠 code-quality change type: code-quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants