Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to GitHub Actions #25

Merged
merged 2 commits into from
Apr 5, 2022
Merged

Switch to GitHub Actions #25

merged 2 commits into from
Apr 5, 2022

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented Feb 15, 2022

CI: https://github.com/Fryguy/bundler-inject/actions?query=branch%3Agithub_actions

Easier to review in 2 commits

The second commit changes all of the tests gems:

The latest rack, which was used as the "gem with no dependencies",
introduced a new dependency, complicating the specs, so this commit
replaces them with gems that following the same pattern, but haven't
been updated in a long time.

  • rack-obama is replaced with rubytest
  • rack is replaced with ansi

The gem versions used in these specs follow a specific pattern in order to
test various versions and overrides. rubytest 0.7.0 depends on ansi >= 0 which
has no dependencies. This allows us to manipulate the various versions of
ansi without introducing additional dependencies complicating the test.
ansi 1.4.2, 1.4.3, 1.5.0 are the three latest versions, and most tests "start"
with ansi 1.4.3 in the base Gemfile. omg 0.0.6 is an additional gem that also
has no dependencies.

@Fryguy
Copy link
Member Author

Fryguy commented Apr 1, 2022

I want to change this one to not hardcode the full 3-digit version number but I can't because of ruby/setup-ruby#304

The latest rack, which was used as the "gem with no dependencies",
introduced a new dependency, complicating the specs, so this commit
replaces them with gems that following the same pattern, but haven't
been updated in a long time.

- rack-obama is replaced with rubytest
- rack is replaced with ansi

The gem versions used in these specs follow a specific pattern in order to
test various versions and overrides. rubytest 0.7.0 depends on ansi >= 0 which
has no dependencies.  This allows us to manipulate the various versions of
ansi without introducing additional dependencies complicating the test.
ansi 1.4.2, 1.4.3, 1.5.0 are the three latest versions, and most tests "start"
with ansi 1.4.3 in the base Gemfile. omg 0.0.6 is an additional gem that also
has no dependencies.
@Fryguy Fryguy changed the title [WIP] Switch to GitHub Actions Switch to GitHub Actions Apr 4, 2022
@Fryguy
Copy link
Member Author

Fryguy commented Apr 4, 2022

@agrare This is ready to go. It's easier to review commit by commit because I had to completely choose new gems to test against.

@agrare agrare merged commit 2b286ba into ManageIQ:master Apr 5, 2022
@Fryguy Fryguy deleted the github_actions branch April 5, 2022 13:35
agrare added a commit that referenced this pull request Jan 9, 2024
Fixed
* Remove remnants of bundler 1.x support (#20)
* Fix detection of bundler version (#21)
* Fix spec issues after release of bundler 2.4.17 (#32)

Added
* Switch to GitHub Actions (#25)
* Add bundler 2.4 to the test matrix (#28)
* Add Ruby 3.1 and 3.2 to test matrix (#29)
* Allow user to specify a PATH to search for gem overrides (#34)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants