SeaQL / sea-schema

🌿 SQL schema definition and discovery
https://docs.rs/sea-schema
Apache License 2.0
192 stars 40 forks source link

Version bump of sea-orm required for compatibility with 0.7.0 #56

Closed WolfSkin0 closed 2 years ago

WolfSkin0 commented 2 years ago

Description

Out of date sea-orm dependency is causing an error involving the Iden trait.

Steps to Reproduce

  1. Use sea-orm 0.7.0 in an entity crate and sea-schema 0.6.0 in a migration crate
  2. Attempt to write a migration, referencing an entity from the aforementioned entity crate

Expected Behavior

The compiler shouldn't throw an error.

Actual Behavior

The compiler reports an incompatibility between sea-orm's Iden and the one that sea-schema knows:

the trait bound `entity::preset::Entity: sea_schema::migration::prelude::Iden` is not satisfied

Reproduces How Often

Always

Versions

└── sea-orm v0.7.0
    ├── sea-orm-macros v0.7.0 (proc-macro)
    ├── sea-query v0.23.0
    │   ├── sea-query-derive v0.2.0 (proc-macro)
    ├── sea-strum v0.23.0
    │   └── sea-strum_macros v0.23.0 (proc-macro)
└── sea-schema v0.6.0
    ├── sea-orm v0.6.0
    │   ├── sea-orm-macros v0.6.0 (proc-macro)
    │   ├── sea-query v0.21.0
    │   │   └── sea-query-derive v0.2.0 (proc-macro) (*)
    │   ├── sea-strum v0.23.0 (*)
    ├── sea-query v0.22.0
    │   └── sea-query-derive v0.2.0 (proc-macro) (*)
    ├── sea-schema-derive v0.1.0 (proc-macro)
WolfSkin0 commented 2 years ago

Closed because it was fixed by 7e3e43beece339fc128dc33000fa391bf8d955d5. Thanks!

billy1624 commented 2 years ago

Thanks for the catch! @WolfSkin0