Run the test suite on GitHub Actions - #1
Conversation
The repository has no CI, so the RSpec suite is only ever run locally and a change that breaks it can be merged without anyone noticing. Add a workflow that runs `rspec` on pushes to master and on every pull request, across Ruby 2.7 through 3.4. Two notes on the approach: * The suite is run without Bundler. Procodile has no runtime dependencies outside the standard library, and the committed Gemfile.lock pins rspec 3.5.x (2016), which does not install on current Rubies. Installing rspec directly keeps the whole matrix green without needing to change or delete the lockfile, which felt out of scope for adding CI. * Ruby 2.7 runs on ubuntu-22.04 because it is not available on the ubuntu-24.04 runner image. Verified locally against Ruby 2.7.8, 3.2.2, 3.3.8 and 3.4.9 (34 examples, 0 failures on each). 3.0 and 3.1 are included in the matrix but were not available locally to test. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01P89F451YDRDBj5br14i3GD
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (1)
Included review availability: 2 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. WalkthroughThe pull request adds a GitHub Actions workflow. The workflow runs RSpec on pushes to ChangesCI testing
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to The workflow adds the intended Ruby test matrix without an established merge-blocking issue. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
1 participant
Fork-side copy of adamcooke/procodile#43, opened here so the workflow actually executes — upstream requires maintainer approval to run Actions on a first-time contributor's fork PR, and that repo has been dormant since 2021.
Validates the Ruby 2.7-3.4 matrix, including 3.0 and 3.1 which weren't available to test locally.
Summary by CodeRabbit