Compile and run will both replace includes now.#248
Conversation
|
@benjie Can you take a look? It seems to look good, IMO. (I hadn't managed to finish my original PR.) |
|
Yeah I'm a bit behind at the moment but it is in my queue 👍 |
|
@benjie thank you very much! |
benjie
left a comment
There was a problem hiding this comment.
Excellent; thanks for your work on this! I've refactored it a bit and made the change to the compile export non-breaking (even though technically it's not a supported interface, there may be people using it).
I use it myself in my data-layer tests. It allows testing against the current.sql easily, without having to commit a migration. |
|
Released in |
Description
compileandrunshould be able to--!includefixtures #217There was already a PR for this, #235, but rather than go two steps removed to get this across the finish line I just forked the up to date repo and made sure to add tests, address the feedback I saw on the other PR.
Performance impact
Security impact
Checklist
yarn lint:fixpasses.yarn testpasses.RELEASE_NOTES.mdfile (if one exists).