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

feat(mongo): add support for migrations in mongo driver #3347

Merged
merged 1 commit into from
Jul 30, 2022

Conversation

B4nan
Copy link
Member

@B4nan B4nan commented Jul 30, 2022

Adds new @mikro-orm/migrations-mongodb package that needs to be explicitly installed. Current CLI commands will work with it transparently.

Mongo migrations have some limitations:

  • no nested transaction support
  • no schema diffing
  • only blank migrations are generated
  • use this.driver or this.getCollection() to manipulate with the database

Closes #295

@codecov-commenter
Copy link

codecov-commenter commented Jul 30, 2022

Codecov Report

Merging #3347 (5ef3a1e) into master (dd04f46) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            master     #3347    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          194       202     +8     
  Lines        12172     12351   +179     
  Branches      2833      2847    +14     
==========================================
+ Hits         12172     12351   +179     
Impacted Files Coverage Δ
packages/core/src/typings.ts 100.00% <ø> (ø)
...ckages/cli/src/commands/MigrationCommandFactory.ts 100.00% <100.00%> (ø)
packages/core/src/drivers/DatabaseDriver.ts 100.00% <100.00%> (ø)
...ges/migrations-mongodb/src/JSMigrationGenerator.ts 100.00% <100.00%> (ø)
packages/migrations-mongodb/src/Migration.ts 100.00% <100.00%> (ø)
...kages/migrations-mongodb/src/MigrationGenerator.ts 100.00% <100.00%> (ø)
packages/migrations-mongodb/src/MigrationRunner.ts 100.00% <100.00%> (ø)
...ackages/migrations-mongodb/src/MigrationStorage.ts 100.00% <100.00%> (ø)
packages/migrations-mongodb/src/Migrator.ts 100.00% <100.00%> (ø)
...ges/migrations-mongodb/src/TSMigrationGenerator.ts 100.00% <100.00%> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd04f46...5ef3a1e. Read the comment docs.

@B4nan B4nan force-pushed the mongo-migrations branch 2 times, most recently from 97c28d3 to ffe84a7 Compare July 30, 2022 19:51
@B4nan B4nan marked this pull request as ready for review July 30, 2022 19:51
Adds new `@mikro-orm/migrations-mongodb` package that needs to be explicitly installed.
Current CLI commands will work with it transparently.

Mongo migrations have some limitations:
- no nested transaction support
- no schema diffing
- only blank migrations are generated
- use `this.driver` or `this.getCollection()` to manipulate with the database

Closes #295
@B4nan B4nan merged commit c5c6115 into master Jul 30, 2022
@B4nan B4nan deleted the mongo-migrations branch July 30, 2022 20:45
@B4nan B4nan restored the mongo-migrations branch January 10, 2023 19:12
@B4nan B4nan deleted the mongo-migrations branch January 10, 2023 19:18
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.

Enable Migrations Support for MongoDB
2 participants