search

db-migrate / node-db-migrate

Database migration framework for node
Other
2.32k stars 360 forks source link

Doesn't support coffeescript #673

Closed ovikholt closed 4 years ago

ovikholt commented 4 years ago

It doesn't seem to support (migrations written in ) coffeescript anymore, even though it's mentioned a few times in your changelog, and you having an options --coffee-file. Any plans to finish it?

There's an especially offensive line in migrations.js: var filesRegEx = /.js$/; but correcting this doesn't alone solve the problem, hehe, interpretation further down the rabbit hole seems to assume js files.

Do I have to implement this myself in order to get my beatiful, beloved coffeescript everywhere? If so, I could use some hints...

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

wzrdtales commented 4 years ago

It should normally still work, but it is an optional component by now. Did you install the coffee plugin in the project using db-migrate?

https://www.npmjs.com/package/db-migrate-plugin-coffee

ovikholt commented 4 years ago

No I hadn't installed that, and it worked!! Thank you!!