diff --git a/.github/workflows/review_pull_request.yml b/.github/workflows/review_pull_request.yml index 1b5f61fe3..31b3a4870 100644 --- a/.github/workflows/review_pull_request.yml +++ b/.github/workflows/review_pull_request.yml @@ -69,30 +69,7 @@ jobs: working-directory: ./template-compose run: ./gradlew koverMergedXmlReport - - name: Set up Ruby - uses: ruby/setup-ruby@v1 - with: - ruby-version: '2.7' - - - name: Cache gems - uses: actions/cache@v2 - with: - path: vendor/bundle - key: ${{ runner.os }}-gems-${{ hashFiles('**/Gemfile.lock') }} - restore-keys: | - ${{ runner.os }}-gems- - - - name: Install Bundle and check environment versions - run: | - echo 'Install Bundle' - bundle config path vendor/bundle - bundle install - echo 'Check environment setup versions' - ruby --version - gem --version - bundler --version - - name: Run Danger + uses: danger/kotlin@1.2.0 env: DANGER_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }} - run: bundle exec danger diff --git a/Dangerfile.df.kts b/Dangerfile.df.kts new file mode 100644 index 000000000..30f51ca56 --- /dev/null +++ b/Dangerfile.df.kts @@ -0,0 +1,27 @@ +import systems.danger.kotlin.* + +danger(args) { + onGitHub { + // Make it more obvious that a PR is a work in progress and shouldn't be merged yet + if (pullRequest.title.contains("WIP")) { + warn("PR is classed as Work in Progress") + } + + // Warn to encourage a PR description + if (pullRequest.body.isBlank()) { + warn("Please provide a summary in the PR description to make it easier to review") + } + + // Warn to encourage that labels should have been used on the PR + if (pullRequest.labels.isEmpty()) { + warn("Please add labels to this PR") + } + } + + onGit { + // Warn when there is a big PR + if (lineOfCode > 500) { + warn("Big PR") + } + } +}