Skip to content
Snippets Groups Projects

Omit Solutions docs from TW review

Merged 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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Peter Leitzen resolved all threads

    resolved all threads

  • Peter Leitzen approved this merge request

    approved this merge request

  • :wave: @splattael, thanks for approving this merge request.

    This is the first time the merge request has been approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Peter Leitzen resolved all threads

    resolved all threads

  • Peter Leitzen enabled an automatic merge when the pipeline for 25207860 succeeds

    enabled an automatic merge when the pipeline for 25207860 succeeds

  • merged

  • Peter Leitzen mentioned in commit eb02e720

    mentioned in commit eb02e720

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading