This class encapsulates our use of the Danger gem.

This reverts merge request !32196

Convert a bunch of Danger rules so they can run
locally.

Marcel Amirault authored 5 years ago and Achilleas Pipinellis committed 5 years ago

Change from ruby mdl to node markdownlint, add
config file to root of project, delete old config
file, update exceptions, and fix one doc that
was didn't meet standards

Signed-off-by: Rémy Coutable <remy@rymai.me>

Signed-off-by: Rémy Coutable <remy@rymai.me>

Toon Claes authored 5 years ago and Nick Thomas committed 5 years ago

See https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/6069

We have another rule that requires squash enabled
when suggestions are applied. No need to fail the
ones that are more than 72 characters long since
they will be squashed anyway.

Toon Claes authored 5 years ago and Mayra Cabrera committed 5 years ago

Stop using two separate lists for the conditions which files require a
database review.

Related discussion:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/30156#note_187732053

If an MR only changes files within `doc/` we can save valueable CI time
by prepending a branch name with `docs-`. This danger rule helps educate
people about that fact.

On the other hand, if someone has a branch with a `docs` prefix or
suffix, the branch shouldn't contain any changes outside of the `docs`
directory