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

misc(migrations) - Add extra check to make sure schema.rb is up to date. #2347

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

nudded
Copy link
Contributor

@nudded nudded commented Jul 30, 2024

Description

Make sure schema.rb is up to date with latest migrations.

@nudded nudded force-pushed the toon-devex-make-sure-schema.rb-uptodate branch 4 times, most recently from c9885ef to dbedbd3 Compare July 30, 2024 14:20
@nudded nudded force-pushed the toon-devex-make-sure-schema.rb-uptodate branch from dbedbd3 to b0775bf Compare July 30, 2024 14:26
@nudded nudded changed the title Draft: misc(migrations) - Add extra check to make sure schema.rb is up to date. misc(migrations) - Add extra check to make sure schema.rb is up to date. Jul 30, 2024
Copy link
Contributor

@julienbourdeau julienbourdeau left a comment

Choose a reason for hiding this comment

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

Interesting. My approach was to check git after the migration were executed.
https://github.com/getlago/lago-api/pull/2272/files

@julienbourdeau
Copy link
Contributor

I love seeing the diff in the Github Action output 👌

@nudded
Copy link
Contributor Author

nudded commented Jul 30, 2024

Interesting. My approach was to check git after the migration were executed. https://github.com/getlago/lago-api/pull/2272/files

I've tried this as well, but by default the schema.rb is not dumped when running the migrations. (And when debugging with act it does not do a checkout, so using simple diff was easier).

@nudded nudded force-pushed the toon-devex-make-sure-schema.rb-uptodate branch from b0775bf to b2f63aa Compare July 31, 2024 06:29
@nudded nudded merged commit 9171d4c into main Jul 31, 2024
6 checks passed
@nudded nudded deleted the toon-devex-make-sure-schema.rb-uptodate branch July 31, 2024 06:40
abdussamadbello pushed a commit to abdussamadbello/lago-api that referenced this pull request Aug 8, 2024
…te. (getlago#2347)

## Description

Make sure schema.rb is up to date with latest migrations.
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