-
Notifications
You must be signed in to change notification settings - Fork 25
Feature: Check commit messages
Commit messages are often converted into changelogs which are often then committed back into repositories.
If a repository has enabled incremental checking, then it wouldn't be terribly hard to check commit messages on PRs (or non-default branches).
This would be optional, and I don't expect it to be on by default (although I may enable it in spell-check-this's workflow).
It's somewhat similar to Feature: Check filenames and paths.
🐛 Flaky in v0.0.21.
See Future work for thoughts on how to improve the implementation.
If you choose to use this feature today, you should either use only_check_changed_files
or include text in advice.md
suggesting that commit message / PR items be added to allow.txt
instead of expect.txt
.
- latest commit
- commits since last push / pr base
- PR description
- PR summary
ℹ️ Note: While PR description and PR summary aren't technically commit messages, there are a number of commit
strategies (especially Squash
) that will end up integrating them into a future commit, and thus it makes sense to offer to check them as well.
Split the check pass for these objects from the main pass. In order to handle them, items really need to be added to allow.txt
, otherwise a commit that adds a term to expect.txt
to accept a commit message will then force the next PRs to remove the newly inserted expect.txt
items.
- Create a branch w/ the selected items as files
- push it
- let check spelling run normally
- Possibly show the git commands to see the individual commit message, e.g.
git show [SHA]
- curl api to get the pr description
- n commits - This is trivial to implement, but without a use case, I don't intend to implement it
- commits since a specific sha or branch - This might be necessary in order to support adding items to
expect.txt
, although I'm convinced it makes more sense to useallow.txt
orpatterns.txt
.