Skip to content

Compact migrations - #266

Open
tvdeyen wants to merge 1 commit into
solidusio:mainfrom
tvdeyen:compact-migrations
Open

Compact migrations#266
tvdeyen wants to merge 1 commit into
solidusio:mainfrom
tvdeyen:compact-migrations

Conversation

@tvdeyen

@tvdeyen tvdeyen commented Jan 3, 2026

Copy link
Copy Markdown
Member

Summary

These migrations are 16 years old. I think it is time to create one migration with all changes consolidated.

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.

@tvdeyen
tvdeyen requested a review from a team January 3, 2026 22:49
@tvdeyen
tvdeyen force-pushed the compact-migrations branch 2 times, most recently from 2330fe0 to 3afd8a1 Compare January 5, 2026 13:33
Some of these migrations are 16 years old.
It is time to create one migration with all
changes consolidated.
@tvdeyen
tvdeyen force-pushed the compact-migrations branch from 3afd8a1 to cbe75a7 Compare August 12, 2026 11:52
@tvdeyen tvdeyen closed this Aug 12, 2026
@tvdeyen tvdeyen reopened this Aug 12, 2026
@tvdeyen

tvdeyen commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

The test failures reminds me of #274 Maybe I tackle this here as well @jarednorman wdyt?

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