diff options
| author | Rémy Coutable <remy@rymai.me> | 2018-05-21 18:41:21 +0200 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2018-07-11 11:52:01 +0200 |
| commit | 5679837cd412c2fb7911dcb33c19e89d8a787db0 (patch) | |
| tree | fe9984ff05042df9ec0fc6de59e85171e337866d /danger/specs | |
| parent | d2ea56a87026de92f31bbcfb360748fcf766b835 (diff) | |
| download | gitlab-ce-5679837cd412c2fb7911dcb33c19e89d8a787db0.tar.gz | |
Start to use Danger for automating MR reviews
Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'danger/specs')
| -rw-r--r-- | danger/specs/Dangerfile | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/danger/specs/Dangerfile b/danger/specs/Dangerfile new file mode 100644 index 00000000000..88e64c57a4b --- /dev/null +++ b/danger/specs/Dangerfile @@ -0,0 +1,13 @@ +# rubocop:disable Style/SignalException + +has_app_changes = !git.modified_files.grep(%r{\A(ee/)?(app|lib|db/(geo/)?(post_)?migrate)/}).empty? +has_spec_changes = !git.modified_files.grep(/spec/).empty? + +if has_app_changes && !has_spec_changes + msg = [ + "You've made some app changes, but didn't add any tests.", + "That's OK as long as you're refactoring existing code (please consider adding the ~backstage label in that case)." + ] + + warn msg.join(" "), sticky: false +end |