Skip to content
This repository has been archived by the owner on Nov 6, 2024. It is now read-only.

Update .pre-commit-config.yaml #179

Merged
merged 1 commit into from
Apr 6, 2024
Merged

Update .pre-commit-config.yaml #179

merged 1 commit into from
Apr 6, 2024

Conversation

AJFSalomon
Copy link
Contributor

Updated .pre-commit-config.yaml with latest urls and revisions for each hook

@AJFSalomon AJFSalomon closed this Apr 5, 2024
@AJFSalomon AJFSalomon reopened this Apr 5, 2024
@AJFSalomon
Copy link
Contributor Author

Just so you know, I reopened the PR without any new changes after accidentally closing it (I naively thought that approval was equivalent to a merge and quickly realized my mistake)

@Czaki
Copy link
Contributor

Czaki commented Apr 5, 2024

We have rule to wait at least 24 hours between accepting and merging to allow other maintainers to put their review.

@kevinyamauchi
Copy link

Thanks for the PR @AJFSalomon !

@AJFSalomon
Copy link
Contributor Author

Thanks for the PR @AJFSalomon !

You are welcome! 😉 By the way, another PR is coming right after this one to update actions versions in tests.

I'm working on a Napari plugin at the moment, so I'm happy to transfer the small improvements I'm making on my own working directory to the cookiecutter project!

@Czaki Czaki merged commit 03540b6 into napari:main Apr 6, 2024
18 checks passed
@AJFSalomon AJFSalomon deleted the update-pre-commit-config branch April 6, 2024 11:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants