Skip to content

Commit

Permalink
doc: adds "PR action: merge" label
Browse files Browse the repository at this point in the history
This label should be placed by the author (or last reviewer if author is not a collaborator) when the PR is complete and ready to merged. This requires the author to explicitly acknolwedge that they are done with the PR and the caretaker is free to merge it. This label brings the CLI caretaking process into alignment with the frameworks and components repos.
  • Loading branch information
dgp1130 committed Dec 10, 2019
1 parent a9b947c commit 5b9d2ea
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 9 deletions.
23 changes: 15 additions & 8 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -110,14 +110,21 @@ Before you submit your Pull Request (PR) consider the following guidelines:
* If we suggest changes then:
* Make the required updates.
* Re-run the Angular DevKit test suites to ensure tests are still passing.
* Rebase your branch and force push to your GitHub repository (this will update your Pull Request):
* Once your PR is approved and you are done with any follow up changes:
* Rebase to the current master to pre-emptively address any merge conflicts.

```shell
git rebase master -i
git push -f
```
* Add the `PR action: merge` label and the correct
[target label](https://github.com/angular/angular/blob/master/docs/TRIAGE_AND_LABELS.md#pr-target)
(if PR author has the project collaborator status, or else the last reviewer
should do this).
* The current caretaker will merge the PR to the target branch(es) within 1-2
business days.

That's it! Thank you for your contribution!
That's it! 🎉 Thank you for your contribution!

#### After your pull request is merged

Expand Down Expand Up @@ -187,14 +194,14 @@ Must be one of the following:

* **build**: Changes that affect the build system or external dependencies. [2]
* **ci**: Changes to our CI configuration files and scripts. [2]
* **docs**: Documentation only changes.
* **docs**: Documentation only changes.
* **feat**: A new feature. [1]
* **fix**: A bug fix. [1]
* **refactor**: A code change that neither fixes a bug nor adds a feature
* **refactor**: A code change that neither fixes a bug nor adds a feature
* **release**: A release commit. Must only include version changes. [2]
* **revert**: A git commit revert. The description must include the original commit message. [2]
* **style**: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc).
* **test**: Adding missing tests or correcting existing tests.
* **style**: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc).
* **test**: Adding missing tests or correcting existing tests.


<sup>[1] This type MUST have a scope. See the next section for more information.</sup><br/>
Expand Down Expand Up @@ -301,12 +308,12 @@ yarn bazel run //etc/api:angular_devkit_core_api.accept
**Note**: In some cases we use aliased symbols to create namespaces.

Example:
```javascript
```javascript
import * as foo from './foo';

export { foo };
```
There are currently not supported by the API guardian.
There are currently not supported by the API guardian.
To overcome this limitation we created `_golden-api.ts` in certain packages.

When adding a new API, it might be the case that you need to add it to `_golden-api.ts`.
2 changes: 1 addition & 1 deletion docs/process/release.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ TBD
## Merging PRs

The list of PRs which are currently ready to merge (approved with passing status checks) can
be found with [this search](https://github.com/angular/angular-cli/pulls?q=is%3Apr+is%3Aopen+review%3Aapproved+status%3Asuccess).
be found with [this search](https://github.com/angular/angular-cli/pulls?q=is%3Apr+is%3Aopen+review%3Aapproved+status%3Asuccess+label%3A%22PR+action%3A+merge%22).
This list should be checked daily and any ready PRs should be merged. For each
PR, check the `PR target` label to understand where it should be merged to. If
`master` is targetted, then click "Rebase and Merge". If the PR also targets a
Expand Down

0 comments on commit 5b9d2ea

Please sign in to comment.