From 4f9a8839a2e7bf0862aba35517407b12595dc1b2 Mon Sep 17 00:00:00 2001 From: Patrik Ragnarsson Date: Mon, 12 Dec 2022 22:20:38 +0100 Subject: [PATCH] Install `libyaml-dev` in CI Needed to be able to install psych >=5. Need to do it before setup-ruby runs as that runs `bundle install`. Related to: - https://github.com/ruby/psych/pull/541 - https://github.com/ruby/setup-ruby/issues/409 - https://github.com/actions/runner-images/issues/6725 Yes, libyaml-dev will be added to GitHub runner images (this week they say) but opening this PR just in case anyone encounters failing builds. I don't think it hurts having this in the repo even after libyaml-dev have been added to the images. Added some blank lines to make the workflow easier to read. --- .github/workflows/test.yml | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 7e026ea9..0f7bcd21 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -16,25 +16,38 @@ jobs: # Due to https://github.com/actions/runner/issues/849, we have to use quotes for '3.0' ruby: [2.6, 2.7, '3.0', 'jruby-9.3'] steps: - - uses: actions/checkout@v2 + - name: Install dependencies + run: | + sudo apt-get install --yes \ + pandoc \ + nodejs \ + pkg-config \ + libxml2-dev \ + libxslt-dev \ + libyaml-dev + + - uses: actions/checkout@v3 + - uses: ruby/setup-ruby@v1 with: ruby-version: ${{ matrix.ruby }} bundler-cache: true - - name: Install dependencies - run: sudo apt-get install -y pandoc nodejs pkg-config libxml2-dev libxslt-dev + - name: Run sinatra tests run: bundle exec rake + - name: Run sinatra-contrib tests working-directory: sinatra-contrib run: | bundle install --jobs=3 --retry=3 bundle exec rake + - name: Run rack-protection tests working-directory: rack-protection run: | bundle install --jobs=3 --retry=3 bundle exec rake + - uses: 8398a7/action-slack@v3 with: status: ${{ job.status }}