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.

Sign up for GitHub

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

Jump to bottom

Added possibility to decorate generated migration files with a custom suffix #1214

Merged
merged 1 commit into from
Jun 13, 2016
Merged

Added possibility to decorate generated migration files with a custom suffix #1214

merged 1 commit into from
Jun 13, 2016

Conversation

motin
Copy link
Contributor

@motin motin commented Jun 9, 2016

It is rather hard to find a certain migration with the current filenames:

PropelMigration_1456744599.php
PropelMigration_1457186710.php
PropelMigration_1457212222.php
PropelMigration_1458157955.php
PropelMigration_1461870141.php
PropelMigration_1462121943.php
PropelMigration_1462860381.php
PropelMigration_1462946617.php
PropelMigration_1463383579.php
PropelMigration_1464338765.php

This PR allows an optional suffix to be specified when creating migrations, so that the filenames directly hint at what the migrations contain:

PropelMigration_1456744599_import_process_data_model_enhancements.php
PropelMigration_1457186710_user_entry_rows_cascade_delete.php
PropelMigration_1457212222_data_model_correction.php
PropelMigration_1458157955_transaction_row_additional_booleans.php
PropelMigration_1461870141_additional_fields.php
PropelMigration_1462121943_input_results_created.php
PropelMigration_1462860381_import_improvements.php
PropelMigration_1462946617_core_mapping_indices.php
PropelMigration_1463383579_correct_utf8_encoding_for_foo_table.php
PropelMigration_1464338765_user_table_definition_correction.php

@marcj marcj merged commit 7515fbc into propelorm:master Jun 13, 2016
@marcj
Copy link
Member

marcj commented Jun 13, 2016

Create addition, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@motin @marcj