Skip to content

Commit

Permalink
[#45] Fix: deprecated actions/setup-ruby usage
Browse files Browse the repository at this point in the history
  • Loading branch information
luongvo committed Dec 5, 2023
1 parent ccb77c7 commit 39ee55d
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 20 deletions.
29 changes: 10 additions & 19 deletions .github/workflows/review_pull_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ on:
pull_request:
types: [ opened, edited, reopened, synchronize ]

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true

jobs:
review_pull_request:
name: Review pull request
Expand Down Expand Up @@ -38,27 +42,14 @@ jobs:
run: ./gradlew jacocoTestReport

- name: Set up Ruby
uses: actions/setup-ruby@v1
uses: ruby/setup-ruby@v1
with:
ruby-version: '2.7'
bundler-cache: true

- 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
- name: Run Danger to wrap up the review
env:
DANGER_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: bundle exec danger
run: |
bundle install
bundle exec danger
6 changes: 5 additions & 1 deletion Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,8 @@ GEM
no_proxy_fix (0.1.2)
nokogiri (1.12.5-x86_64-darwin)
racc (~> 1.4)
nokogiri (1.12.5-x86_64-linux)
racc (~> 1.4)
octokit (4.21.0)
faraday (>= 0.9)
sawyer (~> 0.8.0, >= 0.5.3)
Expand All @@ -93,7 +95,9 @@ GEM
unicode-display_width (2.1.0)

PLATFORMS
universal-darwin-23
x86_64-darwin-19
x86_64-linux

DEPENDENCIES
danger
Expand All @@ -103,4 +107,4 @@ DEPENDENCIES
danger-shroud

BUNDLED WITH
2.2.15
2.2.22

0 comments on commit 39ee55d

Please sign in to comment.