migrate icon indicating copy to clipboard operation
migrate copied to clipboard

Add support for execution of multiple statements in one migration file

Open syndbg opened this issue 8 years ago • 9 comments

  • ~~Authentication functionality,~~ (added in #278)
  • Execution of multiple statements in one migration file

@mattes Just something that prevented us from using migrate initially.

syndbg avatar Sep 15 '17 14:09 syndbg

Note that I'll take a look at the failing tests and add/modify as needed.

syndbg avatar Sep 15 '17 15:09 syndbg

Thanks for your PR. I think the tests are failing somewhere else actually. Need to investigate as well.

mattes avatar Sep 15 '17 17:09 mattes

@JensRantil thanks for your PRs. I closed #280 in favor of this one. I'd much rather find a regex than introducing custom query markup. (Partially related: FAQ:why-two-separate-files)

@syndbg and @JensRantil thanks for bringing this up.

mattes avatar Sep 18 '17 17:09 mattes

Any progress on how to handle strings with newlines? We are considering forking migrate internally and using https://github.com/mattes/migrate/pull/280 because of this.

JensRantil avatar Nov 15 '17 15:11 JensRantil

@JensRantil I totally slept on this PR. Give me a day and I'll finally address everything left.

syndbg avatar Nov 15 '17 16:11 syndbg

@JensRantil @mattes Updated and cleaned up. Cassandra tests still fail and I can't figure out. (note that they were failing before the changes too)

I had troubles getting them running locally, but I got this working eventually. Still I can't find why they're considered a FAIL.

syndbg avatar Nov 15 '17 23:11 syndbg

Any updates on merging this in?

emcfarlane avatar Mar 31 '18 16:03 emcfarlane

@afking It's quite stale. It was ready to merge in November 2017. Idk about now, it's probably obsolete/out-dated.

syndbg avatar Apr 02 '18 09:04 syndbg

@syndbg thanks, i've applied the changes locally.

emcfarlane avatar Apr 02 '18 14:04 emcfarlane