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

Ignore file in migrations folder #756

Open
jasonbronson opened this issue Mar 15, 2019 · 6 comments
Open

Ignore file in migrations folder #756

jasonbronson opened this issue Mar 15, 2019 · 6 comments
Labels

Comments

@jasonbronson
Copy link

Would be great to able to ignore README.md in the migrations folder from the configuration file.

@alexcoroza
Copy link

Would be great to able to ignore README.md in the migrations folder from the configuration file.

Any updates on this one?

@vincentdesmares
Copy link

Also useful for other helpful files like .eslintrc.json. This should be configurable.

@stale
Copy link

stale bot commented Jun 21, 2020

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.

@stale stale bot added the stale label Jun 21, 2020
@stale stale bot closed this as completed Jun 28, 2020
@FlyingPumba
Copy link

Is there any way to workaround this issue? I also have a .eslintrc.json file inside my migrations folder, and its breaking the sequelize db:migrate command.

@dfyz011
Copy link

dfyz011 commented Apr 2, 2024

Any updates?

@WikiRik
Copy link
Member

WikiRik commented Apr 2, 2024

No updates on this. I tried to reproduce it on #1114 but wasn't able to. If you have a reproducible example for this, we can look into this further.

See #1113 for some comments we had about this earlier, including this suggestion for a future feature:
"I think we should look into adding an extensions glob option, and only treat files that match the glob as migrations. It would give the ability to skip files by setting it to something like *.migration.js.

(it would also solve the issue we have with .ts file extensions)"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants