Skip to content

Omit Solutions docs from TW review

Sarah German requested to merge 430270-solutions-docs-no-tws into master

What does this MR do and why?

The Solutions Architecture team is adding content to the Docs site which will not be maintained by the Technical Writing team (see #430270 (closed)).

This MR updates Danger to not prompt MR authors to get a Technical Writing review for pages in this section.

Screenshots or screen recordings

Before After
image image

How to set up and validate locally

  1. Create a test page in the doc/solutions directory: mkdir doc/solutions && echo "# Just Testing" > doc/solutions/index.md
  2. Commit it: git ci doc/solutions -m "Add a test Solutions page"
  3. Run danger: bin/rake danger_local

On this branch, you should not get a prompt to request a Technical Writing review. On the master branch, you will.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #430270 (closed)

Edited by Sarah German

Merge request reports