Skip to content

docs: update contribution merge guide #7611

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
ba3cf52
add issue bot for prs
mtrezza Aug 27, 2021
2af6350
Update CHANGELOG.md
mtrezza Aug 27, 2021
d7a34ce
Update issue-bot.yml
mtrezza Aug 27, 2021
c2f2481
Merge remote-tracking branch 'upstream/master'
mtrezza Aug 27, 2021
60537fe
Merge remote-tracking branch 'upstream/master'
mtrezza Aug 31, 2021
2e69bc7
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 2, 2021
ac221a1
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 2, 2021
23e78ae
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 2, 2021
ab2d9ed
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 3, 2021
59c4214
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 4, 2021
ebefede
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 7, 2021
81165cc
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 7, 2021
a26006e
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 7, 2021
3a3d4fa
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 7, 2021
4f3ad39
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 10, 2021
695f00a
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 10, 2021
7cffb87
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 10, 2021
e0c066e
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 13, 2021
a863c82
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 14, 2021
5610dc5
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 16, 2021
e680e4a
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 24, 2021
6f04834
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 30, 2021
55b7521
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 30, 2021
7857020
Merge remote-tracking branch 'upstream/master'
mtrezza Sep 30, 2021
0b3a4a1
Merge remote-tracking branch 'upstream/master'
mtrezza Oct 10, 2021
ff833d2
Update CONTRIBUTING.md
mtrezza Oct 10, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 16 additions & 11 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,9 @@
- [Wording Guideline](#wording-guideline)
- [Parse Error](#parse-error)
- [Parse Server Configuration](#parse-server-configuration)
- [Commit Message](#commit-message)
- [Pull Request](#pull-request)
- [Breaking Change](#breaking-change)
- [Merging](#merging)
- [Code of Conduct](#code-of-conduct)

## Contributing
Expand Down Expand Up @@ -290,7 +291,7 @@ Introducing new [Parse Server configuration][config] parameters requires the fol
5. Add test cases to ensure the correct parameter value validation. Parse Server throws an error at launch if an invalid value is set for any configuration parameter.
6. Execute `npm run docs` to generate the documentation in the `/out` directory. Take a look at the documentation whether the description and formatting of the newly introduced parameters is satisfactory.

## Commit Message
## Pull Request

For release automation, the title of pull requests needs to be written in a defined syntax. We loosely follow the [Conventional Commits](https://www.conventionalcommits.org) specification, which defines this syntax:

Expand Down Expand Up @@ -325,19 +326,23 @@ Currently, we are not making use of the commit _scope_, which would be written a

### Breaking Change

If a pull request contains a braking change, the description of the pull request must contain a special chapter at the bottom.
If a pull request contains a braking change, the description of the pull request must contain a dedicated chapter at the bottom to indicate this. This is to assist the committer of the pull request to avoid merging a breaking change as non-breaking.

The chapter consists of the phrase `BREAKING CHANGE`, capitalized, in a single line without any formatting. It must be followed by an empty line, then a short description of the breaking change, and ideally how the developer should address it. This chapter should contain more details focusing on the "breaking” aspect of the change, as it is intended to assist the developer in adapting their deployment. However, keep it concise, as it will also become part of the changelog entry.
## Merging

For example:

```
Detailed pull request description...
The following guide is for anyone who merges a contributor pull request into the working branch, the working branch into a release branch, a release branch into another release branch, or any other direct commits such as hotfixes into release branches or the working branch.

BREAKING CHANGE
- For changelog generation, only the commit message set when merging the pull request is relevant. The title and description of the GitHub pull request as authored by the contributor have no influence on the changelog generation. However, the title of the GitHub pull request should be used as the commit message.
- If the pull request contains a breaking change, the commit message must contain the phrase `BREAKING CHANGE`, capitalized and without any formatting, followed by a short description of the breaking change and ideally how the developer should address it, all in a single line. This line should contain more details focusing on the "breaking” aspect of the change and is intended to assist the developer in adapting. Keep it concise, as it will become part of the changelog entry, for example:

The door handle has be pulled up to open the door, not down. Adjust your habits accordingly by walking on your hands.
```
```
fix: remove handle from door

BREAKING CHANGE: You cannot open the door anymore by using a handle. See the [#migration guide](http://example.com) for more details.
```
Keep in mind that in a repository with release automation, merging such a commit message will trigger a release with a major version increment.
- A contributor pull request must be merged into the working branch using `Squash and Merge`, to create a single commit message that describes the change.
- A release branch or the default branch must be merged into another release branch using `Merge Commit`, to preserve each individual commit message that describes its respective change.

## Code of Conduct

Expand Down