Skip to content

Conversation

fohte
Copy link
Contributor

@fohte fohte commented May 21, 2018

This PR adds support for migrating comments for tables with --mysql-change-table-comment option.
This migration is skipped if --ignore-table-comment option is passed.

And this PR fixes an issue about the comment option on create_table.
ActiveRecord 5 allows to specify it, but this option adds the comment for tables not a primary key.

fohte added 3 commits May 21, 2018 17:36
ActiveRecord 5 allows to specify the `comment` option on
`create_table`, but this option adds the comment for tables
not a primary key.
This commit adds support for migrating comments for tables with
`--mysql-change-table-comment` option.

This migration is skipped if `--ignore-table-comment` option is passed.
@coveralls
Copy link

coveralls commented May 21, 2018

Coverage Status

Coverage increased (+0.03%) to 97.193% when pulling bd3e08a on fohte:add-support-table-comment-migration into 83b0d75 on winebarrel:0.7.

@coveralls
Copy link

coveralls commented May 21, 2018

Coverage Status

Coverage increased (+0.03%) to 97.193% when pulling bf09781 on fohte:add-support-table-comment-migration into 83b0d75 on winebarrel:0.7.

1 similar comment
@coveralls
Copy link

coveralls commented May 21, 2018

Coverage Status

Coverage increased (+0.03%) to 97.193% when pulling bf09781 on fohte:add-support-table-comment-migration into 83b0d75 on winebarrel:0.7.

Copy link
Member

@winebarrel winebarrel left a comment

Choose a reason for hiding this comment

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

Thank you!

@winebarrel winebarrel merged commit 334bbec into ridgepole:0.7 May 21, 2018
@fohte fohte deleted the add-support-table-comment-migration branch May 21, 2018 16:12
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.

3 participants