Update pr 8000 e-commit version in documentation#3666
Update pre-commit version in documentation#3666larsoner merged 1 commit intocodespell-project:mainfrom
Conversation
Changed the versioning outlined user documentation for precommit from v2.2.4 to v2.4.1 to reflect modern standards
|
This is my first opensource contribution so feedback on contribution etiquette or standards is greatly appreciated. |
|
While it doesn't sound reasonable to automate such updates, we could add a comment in the pre-commit config file. It would remind us to update the README together with the pre-commit config file: codespell/.pre-commit-config.yaml Lines 70 to 71 in c9cea7c - rev: v2.4.1
+ rev: v2.4.1 # update README.rst accordingly |
|
@larsoner Same problem again with stalled checks displaying "Expected — Waiting for status to be reported". The strange thing is that these stalled checks are sort of duplicates with different full names: the stalled check displays only the the job name According to Github actions: Stuck in "Expected — Waiting for status to be reported", these failures could be caused by There doesn't seem to be a solution other than removing cancel-in-progress.
Unless of course there is a discrepancy between branch protection rules and actual job names, but that would result in consistent errors for each PR, wouldn't it? |
|
The cancel-in-progress would be weird... the jobs actually ran and had the correct names. I have a different more hacky solution that I think should work, I'll open a PR shortly |
Closes: #3662 Update user documentation
Update the version of pre-commit in the documentation from v2.2.4 to v2.4.1 to align with current standards. The previous change was 2 years ago.