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

ruff-pre-commit updating v0.7.4 -> v0.8.6 #390

Merged

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Nov 22, 2024

@cclauss cclauss changed the title ruff-pre-commit updating v0.7.4 -> v0.8.0 ruff-pre-commit updating v0.7.4 -> v0.8.2 Dec 10, 2024
@cclauss
Copy link
Contributor Author

cclauss commented Jan 2, 2025

@nateshmbhat Your review, please.

@cclauss cclauss changed the title ruff-pre-commit updating v0.7.4 -> v0.8.2 ruff-pre-commit updating v0.7.4 -> v0.8.5 Jan 2, 2025
@cclauss cclauss changed the title ruff-pre-commit updating v0.7.4 -> v0.8.5 ruff-pre-commit updating v0.7.4 -> v0.8.6 Jan 4, 2025
Copy link
Owner

@nateshmbhat nateshmbhat left a comment

Choose a reason for hiding this comment

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

lgtm

@nateshmbhat nateshmbhat merged commit fb94674 into nateshmbhat:master Jan 5, 2025
12 checks passed
@cclauss cclauss deleted the pre-commit_autoupdate_2024_11_22 branch January 5, 2025 10:06
@nateshmbhat
Copy link
Owner

Thanks @cclauss . currently i'm working on a flutter project. I am using lefthook for managing git hooks. do u recommend pre-commit framework for managing hooks instead of lefthook for a flutter project ? In lefthook all the tasks need to run manually through commands. some commands are not cross platform.

@cclauss
Copy link
Contributor Author

cclauss commented Jan 5, 2025

pre-commit comes from the Python community so I tend to use it and it works perfectly.

I see husky used occasionally for JavaScript projects but I have never heard of lefthook.

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