Skip to content

Conversation

@AlistairNorman
Copy link
Contributor

No description provided.

Solidus frontend doesn't have newer versions so specifying the branch
for new versions won't work. If the branch is too high it will just
error because it doesn't exist.
@AlistairNorman AlistairNorman force-pushed the alistair/ci-check branch 7 times, most recently from afd6e68 to 1c7d29e Compare October 10, 2025 22:49
@AlistairNorman AlistairNorman force-pushed the alistair/ci-check branch 3 times, most recently from 9600f3f to 51511ce Compare October 10, 2025 23:17
@AlistairNorman AlistairNorman changed the title Check CI Fix CI Oct 10, 2025
@AlistairNorman AlistairNorman force-pushed the alistair/ci-check branch 3 times, most recently from c08e782 to 832d99a Compare October 11, 2025 00:15
Further information can be found in the Github issue.
solidusio/solidus#6326
We were testing some pretty old versions here. In some cases they
weren't compatible with each other.
The version of sqlite we use depends on which version of Rails we're on.
@jarednorman
Copy link
Member

The number of fixup and wip commits in here tells this probably isn't actually ready for review.

@jarednorman
Copy link
Member

Did you manage to force push them out while I was typing that comment??

CleanShot 2025-10-10 at 17 46 46@2x

@AlistairNorman
Copy link
Contributor Author

AlistairNorman commented Oct 11, 2025

It might now actually be ready (if tests all pass)

@jarednorman jarednorman merged commit 7384088 into solidusio-contrib:master Oct 11, 2025
43 checks passed
@jarednorman jarednorman deleted the alistair/ci-check branch October 11, 2025 00:55
@jarednorman
Copy link
Member

:shipit:

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.

2 participants