-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
More examples in Mongo #268
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately, I'm not familiar with MongoDB queries or schema changes, so am not comfortable merging this in without another reviewer that knows MongoDB better.
@zikaeroh @erikdubbelboer If you know MongoDB well enough, could you review this PR?
I have never worked with MongoDB. |
is there any integration test I can do to help? |
We could run the example migrations as part of our tests. e.g. add When |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out TestMigrate()
runs migrations using the examples
Yesterday I struggled to create migrations with some use cases (adding a new field, updating field with a value from another) so I thought of adding these examples as it might help someone and save time :)