chore: update PR template (#6186)
* Update PR template * Make link to checklist bold
This commit is contained in:
Родитель
9594cf8e47
Коммит
26d15abc18
|
@ -1,10 +1,18 @@
|
|||
Checklist for reviewer:
|
||||
## Description
|
||||
|
||||
- [ ] Commits should reference a bug or github issue, if relevant (if a bug is referenced, the pull request should include the bug number in the title).
|
||||
- [ ] If the PR comes from a fork, trigger integration CI tests by running the [Push to upstream workflow](https://github.com/mozilla/bigquery-etl/actions/workflows/push-to-upstream.yml) and provide the `<username>:<branch>` of the fork as parameter. The parameter will also show up
|
||||
in the logs of the `manual-trigger-required-for-fork` CI task together with more detailed instructions.
|
||||
- [ ] If adding a new field to a query, ensure that the schema and dependent downstream schemas have been updated.
|
||||
- [ ] When adding a new derived dataset, ensure that data is not available already (fully or partially) and recommend extending an existing dataset in favor of creating new ones. Data can be available in the [bigquery-etl repository](https://github.com/mozilla/bigquery-etl), [looker-hub](https://github.com/mozilla/looker-hub) or in [looker-spoke-default](https://github.com/mozilla/looker-spoke-default/tree/e1315853507fc1ac6e78d252d53dc8df5f5f322b).
|
||||
<!--
|
||||
Please do not leave this blank
|
||||
This PR [adds/removes/fixes/replaces] the [feature/bug/etc].
|
||||
-->
|
||||
|
||||
For modifications to schemas in restricted namespaces (see [`CODEOWNERS`](https://github.com/mozilla/bigquery-etl/blob/main/CODEOWNERS)):
|
||||
- [ ] Follow the [change control procedure](https://docs.google.com/document/d/1TTJi4ht7NuzX6BPG_KTr6omaZg70cEpxe9xlpfnHj9k/edit#heading=h.ttegrcfy18ck)
|
||||
## Related Tickets & Documents
|
||||
* DENG-XXXX
|
||||
* DSRE-XXXX
|
||||
|
||||
<!--
|
||||
Please reference related Jira tickets, GitHub issues or Bugzilla. This repo has been
|
||||
configured to automatically insert hyperlinks for DSRE and DENG tickets.
|
||||
See https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/managing-repository-settings/configuring-autolinks-to-reference-external-resources
|
||||
-->
|
||||
|
||||
**Reviewer, please follow [this checklist](https://github.com/mozilla/bigquery-etl/.github/reviewer_checklist.md)**
|
|
@ -0,0 +1,10 @@
|
|||
# Checklist for reviewer:
|
||||
|
||||
- [ ] Commits should reference a bug or github issue, if relevant (if a bug is referenced, the pull request should include the bug number in the title).
|
||||
- [ ] If the PR comes from a fork, trigger integration CI tests by running the [Push to upstream workflow](https://github.com/mozilla/bigquery-etl/actions/workflows/push-to-upstream.yml) and provide the `<username>:<branch>` of the fork as parameter. The parameter will also show up
|
||||
in the logs of the `manual-trigger-required-for-fork` CI task together with more detailed instructions.
|
||||
- [ ] If adding a new field to a query, ensure that the schema and dependent downstream schemas have been updated.
|
||||
- [ ] When adding a new derived dataset, ensure that data is not available already (fully or partially) and recommend extending an existing dataset in favor of creating new ones. Data can be available in the [bigquery-etl repository](https://github.com/mozilla/bigquery-etl), [looker-hub](https://github.com/mozilla/looker-hub) or in [looker-spoke-default](https://github.com/mozilla/looker-spoke-default/tree/e1315853507fc1ac6e78d252d53dc8df5f5f322b).
|
||||
|
||||
For modifications to schemas in restricted namespaces (see [`CODEOWNERS`](https://github.com/mozilla/bigquery-etl/blob/main/CODEOWNERS)):
|
||||
- [ ] Follow the [change control procedure](https://docs.google.com/document/d/1TTJi4ht7NuzX6BPG_KTr6omaZg70cEpxe9xlpfnHj9k/edit#heading=h.ttegrcfy18ck)
|
Загрузка…
Ссылка в новой задаче