-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
readme update and PR template and codeowners added
- Loading branch information
1 parent
0046b2a
commit 9a4b617
Showing
4 changed files
with
38 additions
and
8 deletions.
There are no files selected for viewing
Validating CODEOWNERS rules …
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 |
---|---|---|
@@ -0,0 +1,2 @@ | ||
# Require Data team reviews | ||
* @ |
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 |
---|---|---|
@@ -0,0 +1,36 @@ | ||
<!-- If you're making a doc PR or something tiny where the below is irrelevant, delete this | ||
template and use a short description, but in your description aim to include both what the | ||
change is, and why it is being made, with enough context for anyone to understand. --> | ||
|
||
<details> | ||
<summary>PR Checklist</summary> | ||
|
||
### PR Structure | ||
|
||
- [ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs). | ||
- [ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs | ||
otherwise). | ||
- [ ] This PR's title starts with the jira ticket associated with the PR. | ||
|
||
### Thoroughness | ||
|
||
- [ ] This PR adds tests for the most critical parts of the new functionality or fixes. | ||
- [ ] I've updated the docs and the README with the added features, breaking changes, new instructions on how to use the repository. | ||
|
||
### Release planning | ||
|
||
- [ ] I've decided if this PR requires a new major/minor/patch version accordingly to | ||
[semver](https://semver.org/), and I've changed the name of the BRANCH to release/* , feature/* or patch/* . | ||
</details> | ||
|
||
### What | ||
|
||
[TODO: Short statement about what is changing.] | ||
|
||
### Why | ||
|
||
[TODO: Why this change is being made. Include any context required to understand the why.] | ||
|
||
### Known limitations | ||
|
||
[TODO or N/A] |
File renamed without changes.
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