Merge pull request #308 from dblock/danger-pr-comment

Migrate Danger to use danger-pr-comment workflow

Author: dblock

.github/workflows/danger-comment.yml

@@ -0,0 +1,11 @@
+name: Danger Comment
+
+on:
+  workflow_run:
+    workflows: [Danger]
+    types: [completed]
+
+jobs:
+  comment:
+    uses: numbata/danger-pr-comment/.github/workflows/danger-comment.yml@v0.1.0
+    secrets: inherit

.github/workflows/danger.yml

@@ -1,22 +1,13 @@
----
-name: danger
+name: Danger
+
 on:
   pull_request:
     types: [opened, reopened, edited, synchronize]
+
 jobs:
   danger:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v3
-      with:
-        fetch-depth: 0
-    - name: Set up Ruby
-      uses: ruby/setup-ruby@v1
-      with:
-        ruby-version: 3.4
-        bundler-cache: true
-    - name: Run Danger
-      run: |
-        # the personal token is public, this is ok, base64 encode to avoid tripping Github
-        TOKEN=$(echo -n Z2hwX0xNQ3VmanBFeTBvYkZVTWh6NVNqVFFBOEUxU25abzBqRUVuaAo= | base64 --decode)
-        DANGER_GITHUB_API_TOKEN=$TOKEN bundle exec danger --verbose
+    uses: numbata/danger-pr-comment/.github/workflows/danger-run.yml@v0.1.0
+    secrets: inherit
+    with:
+      ruby-version: '3.4'
+      bundler-cache: true

Dangerfile

@@ -1,2 +1,4 @@
+danger.import_dangerfile(gem: 'danger-pr-comment')
+
 changelog.check!
 toc.check!

Gemfile

@@ -21,7 +21,9 @@ group :development, :test do
 end
 
 group :test do
-  gem 'danger-changelog', '~> 0.7.0'
+  gem 'danger', '~> 9.5'
+  gem 'danger-changelog', '~> 0.8.0'
+  gem 'danger-pr-comment', '~> 0.1.0'
   gem 'danger-toc', '~> 0.2.0'
   gem 'minitest'
   gem 'mocha'

test/test_helper.rb

@@ -17,14 +17,13 @@
 
 require 'minitest/autorun'
 require 'minitest/pride'
-require 'minitest/unit'
 require 'mocha/minitest'
 require 'json'
 
 require 'faraday'
 require 'faraday/request/instrumentation'
 
-MiniTest::Assertions.class_eval do
+Minitest::Assertions.class_eval do
   def stub_request(conn, adapter_class = Faraday::Adapter::Test, &stubs_block)
     adapter_handler = conn.builder.handlers.find { |h| h.klass < Faraday::Adapter }
     if adapter_handler