-
Notifications
You must be signed in to change notification settings - Fork 502
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Internal] PR Lint: Fixes to use new title regex (#1699)
* Update PR lint rules * Set the PR Lint version
- Loading branch information
Showing
2 changed files
with
13 additions
and
38 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,43 +1,18 @@ | ||
# Pull Request Template | ||
|
||
## Pull Request Title | ||
1. Changelog will be generated from PR titles. | ||
2. PR titles will be formatted with the following layout. | ||
1. Internal is optional and represents changes with no public facing changes such as test only changes | ||
2. Category represents the area of the change like batch, changefeed, point operation, or query | ||
3. Add or Fix identifies if a new feature is being added or if a bug is being fixed | ||
4. Description is a user friendly explanation of the change | ||
|
||
### Format | ||
`[Internal] Category: (Add|Fix) Description` | ||
|
||
### Example | ||
`Diagnostics: Add GetElapsedClientLatency to CosmosDiagnostics`<br/> | ||
`PartitionKey: Fix null reference when using default(PartitionKey)`<br/> | ||
`[Internal] Query: Add code generator for CosmosNumbers for easy additions in the future.`<br/> | ||
|
||
## Description | ||
|
||
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. | ||
Please include a summary of the change and which issue is fixed. Include samples if adding new API, and include relevant motivation and context. List any dependencies that are required for this change. | ||
|
||
## Type of change | ||
|
||
Please delete options that are not relevant. | ||
|
||
- [ ] Bug fix (non-breaking change which fixes an issue) | ||
- [ ] New feature (non-breaking change which adds functionality) | ||
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) | ||
- [ ] This change requires a documentation update | ||
- [] Bug fix (non-breaking change which fixes an issue) | ||
- [] New feature (non-breaking change which adds functionality) | ||
- [] Breaking change (fix or feature that would cause existing functionality to not work as expected) | ||
- [] This change requires a documentation update | ||
|
||
## Closing issues | ||
|
||
Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such). | ||
|
||
## Assignee | ||
|
||
Please add yourself as the assignee | ||
|
||
## Projects | ||
|
||
Please add relevant projects so this issue can be properly tracked. | ||
|
||
To automatically close an issue: closes #IssueNumber |