Escape the usernames to avoid mentions
Avoid people and groups getting mentioned unintentionally by surrounding the names with backticks.
This commit is contained in:
parent
a8347c22d0
commit
3740bdf21d
1 changed files with 5 additions and 3 deletions
|
@ -1,6 +1,8 @@
|
|||
#### Database Reviewer Checklist
|
||||
|
||||
Thank you for becoming a ~database reviewer! Please work on the list below to complete your setup. For any question, reach out to #database an mention @abrandl.
|
||||
Thank you for becoming a ~database reviewer! Please work on the list
|
||||
below to complete your setup. For any question, reach out to #database
|
||||
an mention `@abrandl`.
|
||||
|
||||
- [ ] Change issue title to include your name: `Database Reviewer Checklist: Your Name`
|
||||
- [ ] Review general [code review guide](https://docs.gitlab.com/ee/development/code_review.html)
|
||||
|
@ -12,7 +14,7 @@ Thank you for becoming a ~database reviewer! Please work on the list below to co
|
|||
- [ ] Read [Understanding EXPLAIN plans](https://docs.gitlab.com/ee/development/understanding_explain_plans.html)
|
||||
- [ ] Review [database best practices](https://docs.gitlab.com/ee/development/#best-practices)
|
||||
- [ ] Review how we use [database instances restored from a backup](https://ops.gitlab.net/gitlab-com/gl-infra/gitlab-restore/postgres-gprd) for testing and make sure you're set up to execute pipelines (check [README.md](https://ops.gitlab.net/gitlab-com/gl-infra/gitlab-restore/postgres-gprd/blob/master/README.md) and reach out to @abrandl since this is currently subject to being changed)
|
||||
- [ ] Get yourself added to [@gl-database](https://gitlab.com/groups/gl-database/-/group_members) group and respond to @-mentions to the group (reach out to any maintainer on the group to get added). You will get TODOs on gitlab.com for group mentions.
|
||||
- [ ] Get yourself added to [`@gl-database`](https://gitlab.com/groups/gl-database/-/group_members) group and respond to @-mentions to the group (reach out to any maintainer on the group to get added). You will get TODOs on gitlab.com for group mentions.
|
||||
- [ ] Make sure you have proper access to at least a read-only replica in staging and production
|
||||
- [ ] Indicate in `data/team.yml` your role as a database reviewer ([example MR](https://gitlab.com/gitlab-com/www-gitlab-com/merge_requests/19600/diffs)). Assign MR to your manager for merge.
|
||||
- [ ] Send one MR to improve the [review documentation](https://about.gitlab.com/handbook/engineering/workflow/code-review/database.html) or the [issue template](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/.gitlab/issue_templates/Database%20Reviewer.md)
|
||||
|
@ -25,7 +27,7 @@ You're all set! Watch out for TODOs on GitLab.com.
|
|||
|
||||
###### Where to go for questions?
|
||||
|
||||
Reach out to `#database` on Slack and mention @abrandl for any questions.
|
||||
Reach out to `#database` on Slack and mention `@abrandl` for any questions.
|
||||
|
||||
cc @abrandl
|
||||
|
||||
|
|
Loading…
Reference in a new issue