2018-08-27 11:26:28 -04:00
# frozen_string_literal: true
2019-02-18 05:03:46 -05:00
docs_paths_to_review = helper . changes_by_category [ :docs ]
2018-08-27 11:26:28 -04:00
unless docs_paths_to_review . empty?
2019-02-11 16:19:06 -05:00
message 'This merge request adds or changes files that require a review ' \
'from the Technical Writing team whether before or after merging.'
2018-08-27 11:26:28 -04:00
markdown ( << ~ MARKDOWN )
2019-02-11 16:19:06 -05:00
## Documentation review
2018-08-27 11:26:28 -04:00
2019-02-11 16:19:06 -05:00
The following files will require a review from the [ Technical Writing ] ( https : / / about . gitlab . com / handbook / product / technical - writing / ) team :
2018-08-27 11:26:28 -04:00
* #{docs_paths_to_review.map { |path| "`#{path}`" }.join("\n* ")}
2019-02-18 16:23:25 -05:00
Per the [ documentation workflows ] ( https : / / docs . gitlab . com / ee / development / documentation / workflow . html ) , the review may occur prior to merging this MR ** or ** after a maintainer has agreed to review and merge this MR without a tech writer review . ( Meanwhile , you are welcome to involve a technical writer at any time 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.)
2019-02-11 16:19:06 -05:00
The technical writer who , by default , will review this content or collaborate as needed is dependent on the [ DevOps stage ] ( https : / / about . gitlab . com / handbook / product / categories / #devops-stages) to which the content applies:
2018-11-09 10:37:22 -05:00
2018-11-16 13:32:14 -05:00
| Tech writer | Stage ( s ) |
| - - - - - - - - - - - - | - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - |
2018-12-06 08:32:17 -05:00
| ` @marcia ` | ~ Create ~ Release + ~ " development guidelines " |
2019-02-11 16:19:06 -05:00
| ` @axil ` | ~ Distribution ~ Gitaly ~ Gitter ~ Monitor ~ Package ~ Secure |
2018-11-16 13:32:14 -05:00
| ` @eread ` | ~ Manage ~ Configure ~ Geo ~ Verify |
| ` @mikelewis ` | ~ Plan |
2018-11-09 13:55:27 -05:00
2019-02-11 16:19:06 -05:00
** To request a review prior to merging **
- Assign the MR to the applicable technical writer , above . If you are not sure which category the change falls within , or the change is not part of one of these categories , mention one of the usernames above .
** To request a review to commence after merging **
2018-12-06 08:32:17 -05:00
2019-02-11 16:19:06 -05:00
- Create an issue for a doc review [ using the Doc Review template ] ( https : / / gitlab . com / gitlab - org / gitlab - ce / issues / new ? issuable_template = Doc % 20 Review ) and assign it to the applicable technicial writer from the table .
2018-08-27 11:26:28 -04:00
MARKDOWN
unless gitlab . mr_labels . include? ( 'Documentation' )
warn 'This merge request is missing the ~Documentation label.'
end
end