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

migration file name to have a datetime as prefixed #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

durgesh-rathod-sf
Copy link

example
pymongo-migrate generate abcd
will create a file named "20230915235959_abcd.py"

example 
pymongo-migrate generate abcd
will create a file named "20230915235959_abcd.py"
@durgesh-rathod-sf durgesh-rathod-sf changed the title migrationn file name to have a datetime as prefixed migration file name to have a datetime as prefixed Sep 15, 2023
@nicon89
Copy link

nicon89 commented Sep 15, 2023

Any idea why I'm receiving notifications about this? I don't work for STX since March...

@MarcinSkrobczynski
Copy link

@nicon89 This is a public repository. Most probably you enabled the notification about this repo (watch or star). Please do not answer here, but write to me directly.

@rooterkyberian
Copy link
Contributor

I'm not a fan of this proposal, since you can get already the same outcome with:
pymongo-migrate generate $(date -u +"%Y%m%d%H%M%S")_abcd
while with your change if someone did not like this prefix they would have hard time overriding it

@RastogiAbhijeet
Copy link

I'm not a fan of this proposal, since you can get already the same outcome with: pymongo-migrate generate $(date -u +"%Y%m%d%H%M%S")_abcd while with your change if someone did not like this prefix they would have hard time overriding it

Maybe someone needs to add this to documentation.

@rooterkyberian
Copy link
Contributor

@RastogiAbhijeet care to be that someone?

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

Successfully merging this pull request may close these issues.

5 participants