Conversation

Copy link Copy Markdown

Member

Pre-commit fail. intentionally didn't actually run black yet so that the changes are better visible - we'll to run it right before merging anyway.

Probably overlaps with #2925 a bit, but that shouldn't be too bad

Copy link Copy Markdown

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 (dev) requirements or the pre-commit hooks. I'm just a friendly reminder to keep the pre-commit hook versions in sync with the dev requirements and the additional dependencies for the hooks in sync with the requirements :)

# Conflicts:
#	pyproject.toml

Copy link Copy Markdown

Choose a reason for hiding this comment

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

Hey there. Relax, I am just a little warning for the maintainers to release directly after merging your PR, otherwise we have broken examples and people might get confused :)

Bibo-Joshi added a commit that referenced this pull request

May 6, 2022

Labels

None yet

Conversation

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Labels

2 participants