Add contributors guide #43
Comments
|
I'll add this in the PR for #38. |
|
bors docs: https://bors.tech/documentation/ |
bors bot
pushed a commit
that referenced
this issue
Jan 2, 2023
Closes #38 and #43. With bors configured in this repository the flow is a bit different, but I think this should be a good setup to allow automating releases. Instead of requiring commit messages to follow the Conventional Commits specification, we just require the PR title to follow it, since that is what ends up being the commit message of the squash merge commit. Overlaps with #47 so closing that one.
bors bot
pushed a commit
that referenced
this issue
Jan 13, 2023
Closes #38 and #43. With bors configured in this repository the flow is a bit different, but I think this should be a good setup to allow automating releases. Instead of requiring commit messages to follow the Conventional Commits specification, we just require the PR title to follow it, since that is what ends up being the commit message of the squash merge commit. Overlaps with #47 so closing that one.
bors bot
pushed a commit
that referenced
this issue
Jan 13, 2023
Closes #38 and #43. With bors configured in this repository the flow is a bit different, but I think this should be a good setup to allow automating releases. Instead of requiring commit messages to follow the Conventional Commits specification, we just require the PR title to follow it, since that is what ends up being the commit message of the squash merge commit. Overlaps with #47 so closing that one.
bors bot
pushed a commit
that referenced
this issue
Mar 13, 2023
Following the main Substrait repository: this makes the PR check match the Merge check by also checking the body of the PR. The PR comment note is moved to the [job summary](https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#adding-a-job-summary). Added a note about the use of [`cargo-smart-release`](https://github.com/Byron/gitoxide/tree/main/cargo-smart-release) to the contributing guide. Closes #55. Closes #43.
bors bot
pushed a commit
that referenced
this issue
Mar 13, 2023
Following the main Substrait repository: this makes the PR check match the Merge check by also checking the body of the PR. The PR comment note is moved to the [job summary](https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#adding-a-job-summary). Added a note about the use of [`cargo-smart-release`](https://github.com/Byron/gitoxide/tree/main/cargo-smart-release) to the contributing guide. Closes #55. Closes #43.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We should add a contributors guide, explaining bors, commit messages, release process, etc
The text was updated successfully, but these errors were encountered: