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

Old migrations need to have ActiveRecord::Migration specified or will raise an error #3099

Open
1 task
carlosdanielpohlod opened this issue Nov 9, 2022 · 5 comments

Comments

@carlosdanielpohlod
Copy link

I'm submitting a ..

  • [ x ] Bug Report
  • Feature Request

Current behavior:
Currently, the specification of ActiveRecord::Migration in old migration files is required but is not specified, so rails db:migrate raises an error

Expected correct behavior:

Steps to reproduce:
run rails db:migrate when database is empty

Other information:

@carlosdanielpohlod
Copy link
Author

I will take this issue to solve

@AndrewKvalheim
Copy link
Member

See #2665 for a script that automates this.

@carlosdanielpohlod
Copy link
Author

See #2665 for a script that automates this.

You filled some files with the correct version, right? the other ones still do not have the version, can I fill in the others?

@AndrewKvalheim
Copy link
Member

Yes, I skipped the migrations that I wasn’t taking the time to review, but ideally all should have the version specified.

@carlosdanielpohlod
Copy link
Author

Yes, I skipped the migrations that I wasn’t taking the time to review, but ideally all should have the version specified.

Ok

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

No branches or pull requests

2 participants