- Sep 06, 2019
-
-
Ash McKenzie authored
This class encapsulates our use of the Danger gem.
-
Ash McKenzie authored
This reverts merge request !32196
-
Ash McKenzie authored
-
- Jun 27, 2019
-
-
Lukas Eipert authored
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
-
- May 30, 2019
-
-
Lin Jen-Shin authored
So it's more modular and extensible
-
- Apr 24, 2019
-
-
Nathan Friend authored
This commit adds a rule to Danger that warns the developer when their MR includes changes to a CE .vue file that has a counterpart in the EE repo or vice-versa.
-
- Apr 04, 2019
-
-
Robert Speicher authored
-
- Mar 29, 2019
-
-
Filipa Lacerda authored
-
- Mar 11, 2019
-
-
Marin Jankovski authored
This extends Danger so it informs MR authors that single codebase merge requests need to be approved by a specific group of engineers. Co-authored-by:
Yorick Peterse <yorickpeterse@gmail.com>
-
- Feb 13, 2019
-
-
Nick Thomas authored
Make danger pick reviewers and maintainers at random, for feontend, backend, database, etc, changes, whenever files belonging to those teams get changed.
-
- Dec 17, 2018
-
-
Lukas Eipert authored
This danger check utilises `yarn-deduplicate` in order to show duplicate dependencies in the yarn.lock dependency tree. Often when introducing new dependencies or updating existing ones, yarn does not seem to build the most optimal dependency tree. In order to prevent those unnecessary dependency updates we are nudging developers and maintainers to resolve these issues in MRs. Automating this with danger especially helps, as yarn.lock files are not that easy to review.
-
- Oct 22, 2018
-
-
Lukas Eipert authored
Danger apparently has three different objects which could contain files you often want to check: - git.added_files - git.modified_files - git.renamed_files The problem: If a file is renamed, `modified_files` contains the file path before the rename. In some Danger checks we use `added_files` + `modified_files`, which might contain the deleted paths of renamed files, but missing the new paths of renamed files. So we need to consider `renamed_files` as well.
-
- Sep 29, 2018
-
-
Lukas Eipert authored
We disabled a lot of eslint rules on a per-file basis. This checks touched files for those and reminds you to re-enable the rules and fix them.
-
- Sep 28, 2018
-
-
Lukas Eipert authored
This adds a Dangerfile which executes `prettier` to find out if someone touched Frontend files and forgot to run it on their current branch.
-
- Sep 10, 2018
-
-
Yorick Peterse authored
This Dangerfile ensures that commit authors write good commit messages. We also prevent the use of Emoji, since these add no value and may not always be displayed properly. Merge commits are currently ignored. In the future we most likely want to stop using these, but this is a separate discussion, hence we ignore them for the time being. The rules enforced by this Dangerfile are based on the article found at https://chris.beams.io/posts/git-commit/. For more information, refer to https://gitlab.com/gitlab-org/gitlab-ce/issues/50003. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/50003
-
- Sep 06, 2018
-
-
Achilleas Pipinellis authored
Every time a doc change is made, a comment to ping the @gl-docsteam is added.
-
- Jul 24, 2018
-
-
gfyoung authored
Checks that new Ruby files will enable frozen string literal. Partially addresses #47424.
-
- Jul 11, 2018
-
-
Rémy Coutable authored
Signed-off-by:
Rémy Coutable <remy@rymai.me>
-