Skip to content

ci: test the Ruby event source - #385

Open
ryanduguid wants to merge 2 commits into
XeroAPI:masterfrom
ryanduguid:agent/test-ruby-event-source
Open

ci: test the Ruby event source#385
ryanduguid wants to merge 2 commits into
XeroAPI:masterfrom
ryanduguid:agent/test-ruby-event-source

Conversation

@ryanduguid

@ryanduguid ryanduguid commented Aug 12, 2026

Copy link
Copy Markdown

Summary

  • test the event source instead of checking out XeroAPI/xero-ruby on every pull request
  • pin both actions and grant the workflow read-only repository access
  • exercise Ruby 3.2 and 3.4 on Linux plus Ruby 3.2 on Windows
  • make Ruby 3.2 the exact package floor declared by companion metadata PR Set the tested Ruby runtime floor #387
  • use a cross-platform Ruby compile check and build the gem after lint/tests
  • bound job runtime while keeping matrix failures independent

Validation

  • actionlint .github/workflows/build-test-lint.yml
  • compiled every Ruby file under lib/ and spec/ with Ruby 3.2.11
  • git diff --check

The full dependency-backed matrix is intentionally delegated to this pull request's exact-head GitHub Actions checks because native Windows gems require MSYS2, which is unavailable in the local portable Ruby environment. Until those hosted checks run, Ruby 3.2 Windows support is unverified rather than passing.

@ryanduguid
ryanduguid marked this pull request as ready for review August 14, 2026 10:28
…oor gap

The pin comment claimed v4.2.2, but 11d5960a326750d5838078e36cf38b85af677262 is
tag v4.4.0. Verified with `gh api repos/actions/checkout/git/ref/tags/v4.2.2`,
which resolves to 11bd71901bbe5b1630ceea73d27597364c9af683, and with
`gh api repos/actions/checkout/commits/11d5960a...`, which is the v4.4.0
"backport fixes to releases-v4" commit dated 2026-07-16.

Corrected the comment rather than repinning to v4.2.2: the pinned SHA is the
newer release, so repinning down would drop roughly 21 months of fixes to keep a
stale comment true. The ruby/setup-ruby pin was checked the same way and its
v1.321.0 comment is correct.

Also recorded the Ruby floor dependency in the matrix. The matrix starts at 3.2
while xero-ruby.gemspec still declares required_ruby_version ">= 2.3", so on its
own this workflow ships a gem advertising 2.3 through 3.1 with no CI coverage
there. PR XeroAPI#387 raises the gemspec floor and should land with or before this.
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.

1 participant