Skip to content

Commit

Permalink
Fix Ruby binding test repo checkout (mmtk#1048)
Browse files Browse the repository at this point in the history
The Ruby binding test now checks out the revision of the `ruby` repo
selected in `mmtk-ruby/mmtk/Cargo.toml` instead of the latest revision.
  • Loading branch information
wks authored Dec 14, 2023
1 parent e55f872 commit bf1bad3
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions .github/workflows/post-review-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -192,12 +192,21 @@ jobs:
path: mmtk-ruby
ref: ${{ needs.binding-refs.outputs.ruby_binding_ref }}

- name: Setup script dependencies
run: ./.github/scripts/ci-setup-script-deps.sh
working-directory: mmtk-ruby

- name: Determine Ruby repository and revision
id: extract-ruby-revision
run: python ./.github/scripts/extract-ruby-revision.py ./mmtk/Cargo.toml --github-output=$GITHUB_OUTPUT
working-directory: mmtk-ruby

- name: Checkout Ruby
uses: actions/checkout@v3
with:
repository: mmtk/ruby
repository: ${{ steps.extract-ruby-revision.outputs.ruby_repo }}
ref: ${{ steps.extract-ruby-revision.outputs.ruby_rev }}
path: ruby
ref: mmtk

- name: Override mmtk-core dependency for binding
run: ./.github/scripts/ci-replace-mmtk-dep.sh ../mmtk-ruby/mmtk/Cargo.toml --mmtk-core-path .
Expand Down

0 comments on commit bf1bad3

Please sign in to comment.