Skip to content

ci: run on ubuntu-24.04 and replace the removed Cloudflare Pages action - #551

Merged
agtang96 merged 2 commits into
mainfrom
ci/fix-runner-images
Oct 25, 2026
Merged

agtang96 merged 2 commits into
mainfrom
ci/fix-runner-images

Conversation

@agtang96

Copy link
Copy Markdown
Contributor

Description

Three CI checks currently fail on every PR before running anything (seen on #550):

Check Error Cause
Ruby 3.1 Unavailable version 3.1.7 for ruby on ubuntu-26.04 ubuntu-latest is now ubuntu-26.04, and ruby-builder has no Ruby 3.0 or 3.1 builds for it
test (Rubocop) 404 downloading ruby-3.0.0-ubuntu-26.04.tar.gz same, plus a setup-ruby pinned to an old SHA
docs Unable to resolve action cloudflare/pages-action, not found the action's repository is gone

Changes:

  • All three workflows: runs-on: ubuntu-24.04. Ruby 3.0.7 and 3.1.7 are prebuilt for 24.04.
  • Rubocop and sdk-reference: setup-ruby moves from the old pinned SHA to the v1.328.0 SHA. ruby-version: '3.0' is quoted so it means 3.0, not the YAML float 3.
  • sdk-reference: cloudflare/pages-action@v1 becomes cloudflare/wrangler-action (v4.1.3, pinned by SHA). It runs pages deploy doc --project-name=nylas-ruby-sdk-reference --branch=<same branch as before>, uses the same two secrets, and the environment URL now comes from the deployment-url output.

Check names are unchanged, so the required checks (Ruby 3, Ruby 3.1, Ruby 3.2, test) still match.

Checked locally in Docker:

  • Ruby 3.0.7 (default Bundler 2.2.33, no lockfile): rspec passes with 320 examples, rubocop reports no offenses across 80 files, and yard builds.
  • Ruby 3.1.7: rspec passes with 320 examples.

The Pages deploy step can only be verified by this PR's own docs run.

Not changed, worth a follow-up: in rspec.yml the matrix entry ruby: 3.0 is a YAML float, so the "Ruby 3" job runs the latest 3.x, not 3.0. Quoting it would rename the check to "Ruby 3.0", and the branch protection rule would then need updating, so I left it.

License

I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CEuUhrasuRFEFW6KcTW7QN

ubuntu-latest moved to ubuntu-26.04, which has no prebuilt Ruby 3.0 or
3.1. That breaks the Ruby 3.1 and Rubocop jobs. Pin the runner to
ubuntu-24.04, and move the old pinned setup-ruby SHA to v1.328.0.

cloudflare/pages-action can no longer be resolved, which breaks the docs
job. Deploy with cloudflare/wrangler-action and the equivalent
`pages deploy` command.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CEuUhrasuRFEFW6KcTW7QN
Comment thread .github/workflows/sdk-reference.yaml Fixed
@agtang96
agtang96 requested a review from qasim-nylas October 25, 2026 03:07

@qasim-nylas qasim-nylas left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CEuUhrasuRFEFW6KcTW7QN
@agtang96
agtang96 merged commit 81f578a into main Oct 25, 2026
8 checks passed

This branch was successfully deployed

1 active deployment
sdk-reference — 90268188 Deployed Oct 25, 2026 by agtang96 via docs #333
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants