Skip to content

Conversation

@wesrich
Copy link
Contributor

@wesrich wesrich commented Nov 5, 2025

Ensure CI runs on GH Actions with the matrix of supported Ruby/ActiveSupport versions. We may end up wanting to add RSpec in there somewhere, but for now....

@wesrich wesrich requested review from Braden-077 and Copilot November 5, 2025 17:30
@wesrich wesrich self-assigned this Nov 5, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the CI infrastructure by migrating from Travis CI to GitHub Actions and adds support for testing against multiple ActiveSupport versions. Key changes include:

  • Removed Travis CI configuration and added GitHub Actions workflow with lint and test jobs
  • Created gemfiles for testing against ActiveSupport 7.1, 7.2, 8.0, and 8.1
  • Updated test configuration to output JUnit XML reports and changed a skipped test from xit to it with skip metadata

Reviewed Changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/workflows/ci.yml New GitHub Actions CI workflow with RuboCop linting and matrix testing across Ruby and ActiveSupport versions
.travis.yml Removed Travis CI configuration
.ruby-version Set Ruby version to 3.4.7
.rspec Added RSpec JUnit formatter output
.gitignore Added pattern to ignore built gem files
Gemfile Added rubocop and rspec_junit_formatter gems for development/testing
gemfiles/activesupport-* Created gemfiles for testing against multiple ActiveSupport versions
spec/marsh_grass_spec.rb Changed slow test from xit to use skip metadata instead

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wesrich wesrich force-pushed the github-actions branch 2 times, most recently from 5f532f5 to d4a43d0 Compare November 5, 2025 17:35
@wesrich wesrich marked this pull request as ready for review November 5, 2025 17:41
@wesrich wesrich merged commit a81e266 into main Nov 7, 2025
17 checks passed
@wesrich wesrich deleted the github-actions branch November 7, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants