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

Combine migrations for solid_cache_entries #172

Merged
merged 1 commit into from
Aug 20, 2024
Merged

Combine migrations for solid_cache_entries #172

merged 1 commit into from
Aug 20, 2024

Conversation

djmb
Copy link
Collaborator

@djmb djmb commented May 1, 2024

Combine the migrations for solid_cache_entries into a single file.

@djmb djmb added the v1.0 label May 1, 2024
@PikachuEXE
Copy link

Should this gem be used in production before 1.0 then?
No idea what steps are required to migrate from 0.x to 1.0

@akhilgkrishnan
Copy link
Member

Should this gem be used in production before 1.0 then? No idea what steps are required to migrate from 0.x to 1.0

We probably wanna consider some migrating plan, before the release.

@djmb
Copy link
Collaborator Author

djmb commented Aug 12, 2024

The migrations were updated in v0.4 - see https://github.com/rails/solid_cache/blob/main/upgrading_to_version_0.4.x.md.

Anyone at a lower version needs to first go to v0.4 and then follow those steps. Anyone who has done that will be able to upgrade to v1.0 without issues - the combined migration is not needed and will not be installed because they've already got a matching one.

@djmb djmb force-pushed the combine-migrations branch from 82f257c to ef837f0 Compare August 13, 2024 14:13
Combine the migrations for solid_cache_entries into a single file. Spot
if someone is migrating directly from v0.3 or earlier and raise an error.
@djmb djmb force-pushed the combine-migrations branch from ef837f0 to af4d016 Compare August 20, 2024 13:07
@djmb djmb merged commit e4806ec into main Aug 20, 2024
19 checks passed
@djmb djmb deleted the combine-migrations branch August 20, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants