Skip to content

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Feb 13, 2025

Summary

Run this extensions own specs on GH actions and generate new extensions to run their specs on GH as well.

Drops CI builds for Solidus 4.1 and 4.2 due to the fact that Rails 7.0 cannot be installed anymore, because of
bug caused by the concurrent-ruby gem which has been fixed and merged but not released as gem and never will, because Rails 7.0 is out of support and will reach EOL on April, 1 2025.

This adds builds against Solidus v4.5, main (4.6.0.a) and Rails 8.0 so that we can test extensions with upcoming Solidus releases.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

  • 📖 I have updated the README to account for my changes.
  • 📑 I have documented new code with YARD.
  • 🛣️ I have opened a PR to update the guides.
  • ✅ I have added automated tests to cover my changes.
  • 📸 I have attached screenshots to demo visual changes.

@mergify
Copy link
Contributor

mergify bot commented Feb 13, 2025

It looks like this PR is missing a label to determine the type of change it introduces. The maintainer should add one of the following labels:

  • bug for bugfixes.
  • enhancement for new features and improvements.
  • documentation for documentation changes.
  • security for security patches.
  • removed for feature removals.
  • infrastructure for internal changes that should not go in the changelog.

Additionally, the maintainer may also want to add one of the following:

  • breaking for breaking changes.
  • deprecated for feature deprecations.

Once the correct labels have been set, simply remove the needs changelog label label from this PR so I can merge it.

@mergify mergify bot added the needs changelog label Needs a label to determine the type of change. label Feb 13, 2025
@tvdeyen tvdeyen added enhancement Improves an existing feature. and removed needs changelog label Needs a label to determine the type of change. labels Feb 13, 2025
@tvdeyen tvdeyen force-pushed the use-github-actions branch 7 times, most recently from 06c6ee3 to 5884f1f Compare February 14, 2025 10:58
@tvdeyen tvdeyen enabled auto-merge February 14, 2025 11:08
tvdeyen and others added 5 commits February 26, 2025 17:24
We want to migrate away from circle ci
Rails 7.0 is untestable because the rails _7.0_ new command
fails because of a bug that has not been fixed in the published
rails gem and wont be ever. Since rails 7.0 will be eol in 6 weeks
it does not make sense to waste carbon on it.
The sandbox wants to install Solidus and migrate before running the
install generator. That's bad, because migrating requires the app to
boot, and booting fails because Sprockets fails with an error if the
`manifest.js` file is not present.

Since the file is quite simple, we can just pass it into a heredoc. This
is needed for Rails 8, where the install generator won't create the file
for us.
@tvdeyen tvdeyen disabled auto-merge February 26, 2025 16:53
@tvdeyen tvdeyen merged commit 650d32e into main Feb 26, 2025
50 checks passed
@tvdeyen tvdeyen deleted the use-github-actions branch February 26, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improves an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants