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

MikroORM v3 #80

Merged
merged 185 commits into from Nov 20, 2019
Merged

MikroORM v3 #80

merged 185 commits into from Nov 20, 2019

Conversation

B4nan
Copy link
Member

@B4nan B4nan commented Jul 25, 2019 •
edited

Progress

Documentation

  • autoFlush
  • new schema generator API
  • entity generator
  • CLI
  • virtual properties
  • new naming strategy api (getClassName())
  • new logger configuration
  • Reference<T> wrapper
  • MariaDB driver
  • Read replica connections
  • Cross-database joins and cross-schema joins
  • Defining relationships
  • Upgrading v2 to v3 guide - autoFlush, knex, fk option, ...
  • Reworked entity definition
  • Deployment section
  • Migrations
  • Update example repositories, maybe split them to dedicated repos Simple example project in v3 #131
  • Metadata providers
  • Update roadmap

docs for v3 can be found here: https://mikro-orm.github.io/docs/v3/

upgrading guide:

@B4nan B4nan added this to the 3.0 milestone Jul 25, 2019
@coveralls
Copy link

coveralls commented Jul 25, 2019 •
edited

Pull Request Test Coverage Report for Build 684

  • 1150 of 1150 (100.0%) changed or added relevant lines in 61 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%
Totals Coverage Status
Change from base Build 643: 0.0%
Covered Lines: 3227
Relevant Lines: 3227
💛 - Coveralls

@B4nan B4nan self-assigned this Aug 6, 2019
@B4nan B4nan mentioned this pull request Aug 7, 2019
Closed
@B4nan B4nan force-pushed the dev branch 2 times, most recently from 9584059 to a08722b Compare August 7, 2019 13:11
@lgtm-com
Copy link

lgtm-com bot commented Aug 7, 2019

This pull request introduces 1 alert when merging a08722b into 972c6e4 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@B4nan B4nan force-pushed the dev branch 5 times, most recently from 8df2917 to 9b7cbb2 Compare August 9, 2019 22:22
codeclimate[bot]
codeclimate bot reviewed Aug 14, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 5466 lines exceeds the maximum allowed for the inline comments feature.

@lgtm-com
Copy link

lgtm-com bot commented Aug 14, 2019

This pull request introduces 4 alerts when merging 0b49a8e into 1f590c4 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

codeclimate[bot]
codeclimate bot reviewed Aug 14, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 5466 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 15, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 5461 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 16, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 6550 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 16, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 6557 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 16, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 6567 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 17, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 7301 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 19, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 8008 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 19, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 7876 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Aug 20, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 7980 lines exceeds the maximum allowed for the inline comments feature.

@lgtm-com
Copy link

lgtm-com bot commented Nov 14, 2019

This pull request introduces 2 alerts when merging 3157572 into 074a145 - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@mikro-orm mikro-orm deleted a comment from lgtm-com bot Nov 14, 2019
@lgtm-com
Copy link

lgtm-com bot commented Nov 14, 2019

This pull request introduces 2 alerts when merging babcc52 into 074a145 - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@lgtm-com
Copy link

lgtm-com bot commented Nov 15, 2019

This pull request introduces 2 alerts when merging 357ff4b into 074a145 - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@lgtm-com
Copy link

lgtm-com bot commented Nov 15, 2019

This pull request introduces 2 alerts when merging ad4cd16 into 074a145 - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@B4nan
feat(schema): add support for create/drop database (#237)
6e58332 
- `schema:create` now automatically tries to create the database if it not exists
- `schema:drop` now has additional flag `--drop-db` that allows to drop the whole database instead of just dropping its tables
@lgtm-com
Copy link

lgtm-com bot commented Nov 18, 2019

This pull request introduces 2 alerts when merging 6e58332 into 074a145 - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@lgtm-com
Copy link

lgtm-com bot commented Nov 18, 2019

This pull request introduces 2 alerts when merging aea3614 into 074a145 - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 2 alerts when merging 908aba2 into ad9103e - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@B4nan
chore: update mongodb to 3.3 and use new unified topology (#241)
f006c3f 
Closes #112
@B4nan
feat(metadata): add ReflectMetadataProvider, rename the ts-morph …
d740eb3 
…one (#240)

- `TypescriptMetadataProvider` is now renamed to `TsMorphProvider`.
- `ReflectMetadataProvider` is provided that does not use `ts-morph` but have some limitations on entity definition

Closes #235
@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 2 alerts when merging d740eb3 into ad9103e - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 2 alerts when merging 04b0fd2 into ad9103e - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@B4nan
chore: move typescript to dev dependencies
a5d8a78 
TS will be still part of runtime dependencies because of ts-morph, this is more of a change to remove some confusion as ts-morph is what we depend on.
@B4nan
chore: improve upgrading docs
8232a98
@lgtm-com
Copy link

lgtm-com bot commented Nov 20, 2019

This pull request introduces 2 alerts when merging 8232a98 into ad9103e - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@lgtm-com
Copy link

lgtm-com bot commented Nov 20, 2019

This pull request introduces 2 alerts when merging d088efd into ad9103e - view on LGTM.com

new alerts:

  • 1 for Unreachable method overloads
  • 1 for Incomplete string escaping or encoding

@B4nan B4nan merged commit d4eddfd into master Nov 20, 2019
@B4nan
Copy link
Member Author

B4nan commented Nov 20, 2019

Time has come, merged and released 3.0.0-rc.

Will keep it in RC till new website (docusaurus rewrite) and announcement article are ready. Hopefully stable release will be out early december.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codeclimate codeclimate[bot] codeclimate left review comments

Assignees

@B4nan B4nan

Labels
None yet
Projects
None yet
Milestone
3.0
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@B4nan @coveralls @Ty3uK @lookfirst @pepakriz @jsprw