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 for testing #1198

Merged
merged 6 commits into from
Sep 16, 2022

Conversation

hibachrach
Copy link
Contributor

@hibachrach hibachrach commented Aug 26, 2022

Converts from Travis to GH actions. Fixes #1193

@hibachrach hibachrach force-pushed the switch-to-github-actions branch 3 times, most recently from c9c7d20 to efbf304 Compare August 26, 2022 05:30
@hibachrach hibachrach changed the title TEMP Switch to GitHub Actions for testing Switch to GitHub Actions for testing Aug 26, 2022
@hibachrach
Copy link
Contributor Author

Ah I think #1150 broke some of the tests a bit more. The new version of webpack-dev-server has a bug because of an undeclared dependency (webpack/webpack-dev-server#2396) and changed the peer dependency support to be webpack@^4.0.0. Not sure how to proceed

@hibachrach
Copy link
Contributor Author

I do think we ought to revisit the supported versions of Webpacker with this gem. I'm generally new to this sort of thing so unsure of where to cut things off (especially considering non-Shakapacker Webpacker is already retired).

@justin808
Copy link
Collaborator

@HarrisonB I agree with you. I opened an issue for this: #1199.

@justin808
Copy link
Collaborator

justin808 commented Sep 9, 2022

@HarrisonB any update? I mentioned that we can revert the new webpack-dev-server until we get CI running.

See here for an argument as to why to do so
Issue was introduced in

minitest/minitest@6fecff9

and hack workaround was introduced in

minitest/minitest@b8ddc4f

I *think* that changing the API make this work w/o the workaround would
require a breaking change so we can leave that for later.
This was EOLed over a year ago and doing a `bundle install` with the
current set of packages gives

> minitest-5.16.3 requires ruby version < 4.0, >= 2.6, which is
> incompatible with the current version
Unfortunately Bundler 2 and JRuby 9.1 are not compatible on GitHub
actions (see here: ruby/setup-ruby#108) which
leads to failures.

Additionally, due to the `webdrivers` gem as a testing dependency
needing upgrading, 9.2 is also a no-go.

JRuby 9.3 matches (MRI) Ruby 2.6 which we just made the floor of the testing
matrix, so this is likely a good version to match anyhow.
@justin808
Copy link
Collaborator

@HarrisonB let me know when to review.

@hibachrach hibachrach marked this pull request as ready for review September 14, 2022 20:51
@hibachrach
Copy link
Contributor Author

@justin808 It's now ready for review

Copy link
Collaborator

@justin808 justin808 left a comment

Choose a reason for hiding this comment

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

LGTM

@justin808 justin808 merged commit f873861 into reactjs:master Sep 16, 2022
@justin808
Copy link
Collaborator

Thanks, @hibachrach. Let's see if this runs! We might need some followups.

@justin808
Copy link
Collaborator

Looks like it's running: https://github.com/reactjs/react-rails/actions

@hibachrach hibachrach deleted the switch-to-github-actions branch September 17, 2022 00:23
hibachrach added a commit to hibachrach/react-rails that referenced this pull request Sep 17, 2022
@ahangarha ahangarha mentioned this pull request Nov 14, 2022
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.

Convert from Travis to Github Actions
2 participants