Add multi db support for migrations #1
Conversation
Ciebiada
left a comment
Ciebiada
left a comment
There was a problem hiding this comment.
Overall looks good, but left some comments :D
MikolajSiwek
requested review from
Ciebiada,
czereba and
swiatek
and removed request for
swiatek
Ciebiada
left a comment
Ciebiada
left a comment
There was a problem hiding this comment.
Looks good! Some minor comments
step as schema updates will be passed in a PR's, update README
| migrator(migration.version).run | ||
| end | ||
|
|
||
| Rake::Task["db:schema:dump"].invoke if ActiveRecord::Base.schema_format == :ruby |
There was a problem hiding this comment.
@czereba @Ciebiada to bodajże to miejsce odpowiedzialne za tworzenie schema.
Z tego co pamiętam to usuneliśmy to, bo stwierdziliśmy że tego gema używamy i tak głównie przy deploymencie a wtedy schema już jest zupdatowana bo puszczamy migrację normalnie lokalnie updatując wtedy scheme 🤔
(Jest w PR już zupdatowana)
Jak są railsy to pewnie można używać railsowej komendy do migracji i ona nie używa gema ... ale dla projektów quasi rails hmm pewnie trzeba by użyć bundle exec rake db:migrate i wtedy wszystko się powinno zadziać okay ... bo tutaj puszczając lokalnie pewnie nie musimy puszczać w grupie? 🤔
Description
Grouping for migrations in a multi database environment was not working correctly as this gem does not support that Rails 6+ feature. Handling multidb envs was added to this version of the gem.
How Has This Been Tested?
Tested locally on socialguide and mjollnir
Please describe how you tested your changes,
see how your change affects other areas of the code, etc.
Pull Request Checklist
Change Management
https://issues.talkwalker.net/issue/SCR-7023