Skip to content

Conversation

@ayushjariyal
Copy link
Contributor

issue #2341

In this PR i update only pre-commit-hooks (updating v5.0.0 -> v6.0.0) in .pre-commit-config.yaml file.

i tested it with 'make lint' command and it looks good to me.
image

@ayushjariyal
Copy link
Contributor Author

@Fokko , Could you please let me know if any further changes are needed, or should I go ahead and proceed with bumping the other hooks?

@Fokko
Copy link
Contributor

Fokko commented Aug 20, 2025

Thanks @ayushjariyal This looks great.

Could you please let me know if any further changes are needen

It looks like there are no changes required for bumping this hook. I will suspect other hooks (eg ruff, or mypy) to require changes to the code.

@Fokko Fokko mentioned this pull request Aug 20, 2025
@Fokko Fokko merged commit 722b8bd into apache:main Aug 20, 2025
10 checks passed
@Fokko
Copy link
Contributor

Fokko commented Aug 20, 2025

Moving this forward, thanks @ayushjariyal 🙌

@ayushjariyal ayushjariyal deleted the bump-pre-commit branch August 21, 2025 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants