core-validate-commit
core-validate-commit copied to clipboard
nodejs
Validate commit messages for Node.js core
[This PR](https://github.com/nodejs/node/pull/49730) was merged with a non-imperative verb starting the commit message: `benchmark: added new benchmarks for blob`. [From what the docs says](https://github.com/nodejs/node/blob/main/doc/contributing/pull-requests.md#commit-message-guidelines), it should be a invalid message. The...
fixes #69 This PR is for adding in tests for the commands of the cli. It is currently a WIP, and more commits will be added to it. Probably 1...
When an invalid subsystem is encountered, attempt to make a suggestion based on the nearest match (if any) from the valid subsystems. Example: ```console ✔ 0:0 skipping fixes-url fixes-url ✔...
Add instructions and a hook to call core-validate-commit from git commits. Depends on https://github.com/evanlucas/core-validate-commit/pull/11 Known issues: - leaves ${TMPF} around (temporary file) - probably will reject `fixup:` type commit logs...
add the usage model of `core-validate-commit file:///somefile…` The file needs to approximate the output of `git show --quiet --format=medium ${sha}` Semver: minor
tl;dr: I'd like to open a PR to replace the `node_js` section in the `.travis.yml` with an imported https://github.com/nodejs/ci-config-travis/blob/master/lts/gte-8.yml Background: [Package Maintenance WG](https://github.com/nodejs/package-maintenance) has built a repository of [shared Travis...
While working on issue #69 I started to come across places where it was a little difficult to test if just a particular flag was called. I think it might...
When i was adding in the tests for the `--list-subsystem` flag, i noticed that there were no tests for the other command line flags. Now that there is a test...
Per the [collaborator guide](https://github.com/nodejs/node/blob/master/COLLABORATOR_GUIDE.md#reverting-commits): > Commits are reverted with git revert , or git revert .. for multiple commits. Commit metadata and the reason for the revert should be appended....
