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

Docs: make migrating Vagrant box command a one-liner #642

Merged
merged 1 commit into from Nov 10, 2015

Conversation

tjwudi
Copy link
Contributor

@tjwudi tjwudi commented Nov 9, 2015

No description provided.

@tjwudi tjwudi mentioned this pull request Nov 9, 2015
PYTHONPATH=. bin/run python migrations/0011_migrate_bigquery_to_json.py
PYTHONPATH=. bin/run python migrations/0012_add_list_users_permission.py
PYTHONPATH=. bin/run python migrations/0013_update_counter_options.py
export PYTHONPATH=. && find migrations/ -type f | grep 00 --null | xargs -I file bin/run python file
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it guaranteed that find will return files in their order? (01 before 02, etc)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for those with number prefix, yes :)

@arikfr arikfr changed the title Make migrating command one-liner Docs: make migrating Vagrant box command a one-liner Nov 10, 2015
arikfr added a commit that referenced this pull request Nov 10, 2015
Docs: make migrating Vagrant box command a one-liner
@arikfr arikfr merged commit 8408409 into getredash:master Nov 10, 2015
@arikfr
Copy link
Member

arikfr commented Nov 10, 2015

Merged. Thanks!

@tjwudi
Copy link
Contributor Author

tjwudi commented Nov 10, 2015

Np! FYI, I don't think it can be a sustainable solution to maintain migrations order manually by adding numbered prefixes. I think at some point we can use some other migration tools that persist migration histories in database like Django's.

@arikfr
Copy link
Member

arikfr commented Nov 11, 2015

Totally agree.

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.

None yet

2 participants