diff options
-rw-r--r-- | danger/documentation/Dangerfile | 31 |
1 files changed, 21 insertions, 10 deletions
diff --git a/danger/documentation/Dangerfile b/danger/documentation/Dangerfile index f5923a22af9..422f2cb2551 100644 --- a/danger/documentation/Dangerfile +++ b/danger/documentation/Dangerfile @@ -24,19 +24,30 @@ The following files require a review from the Documentation team: * #{docs_paths_to_review.map { |path| "`#{path}`" }.join("\n* ")} -To make sure these changes are reviewed, mention a technical writer in a separate -comment, and explain what needs to be reviewed. Please don't mention them -until your changes are ready for review. +When your content is ready for review, mention a technical writer in a separate +comment and explain what needs to be reviewed. -Who to ping [based on DevOps stages](https://about.gitlab.com/handbook/product/categories/#devops-stages): +You are welcome to mention them sooner if you have questions about writing or updating +the documentation. GitLabbers are also welcome to use the [#docs](https://gitlab.slack.com/archives/C16HYA2P5) channel on Slack. -- `@axil`: ~Distribution ~Packaging ~Monitoring ~Secure ~Gitaly ~Gitter -- `@eread`: ~Manage ~Geo ~Verify ~Configure -- `@marcia`: ~Create ~Release -- `@mikelewis`: ~Plan +Who to ping [based on DevOps stages](https://about.gitlab.com/handbook/product/categories/#devops-stages): -If you don't know who to ping, or your change doesn't fall under one of the above -categories, you can mention the whole team with `@gl-docsteam`. +| ~Create | @`marcia` | +| ~Configure | `@eread | +| ~Distribution | `@axil` | +| ~Geo | `@eread | +| ~Gitaly | `@axil` | +| ~Gitter | `@axil` | +| ~Manage | `@eread | +| ~Monitoring | `@axil` | +| ~Packaging | `@axil` | +| ~Plan | `@mikelewis | +| ~Release | `marcia` | +| ~Secure | `@axil` | +| ~Verify | `@eread | + +If you are not sure which category the change falls within, or the change is not +part of one of these categories, you can mention the whole team with `@gl-docsteam`. MARKDOWN unless gitlab.mr_labels.include?('Documentation') |