-
Notifications
You must be signed in to change notification settings - Fork 138
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
Fix usage of dependencySatisfies with a peerDependency #794
Conversation
Similar fix in |
How can we get this merged and released? This is blocking a lot of my pipelines.. |
I approved the CI run, but it seems that it got cancelled; I'm restarting it |
@rwjblue probably it cancels 3.x tests because release/beta get fail too fast at build time because of ember-auto-import v2/webpack miss which is another level of effort |
packages/ember-engines/package.json
Outdated
@@ -86,7 +86,7 @@ | |||
"ember-addon" | |||
], | |||
"dependencies": { | |||
"@embroider/macros": "^0.43.5", | |||
"@embroider/macros": "^0.50.2", |
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.
@anehx @embroider/macros
is now 1.0.0
so we should be able to do just "@embroider/macros": "^1.0.0"
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.
Done!
565e2f2
to
c08f765
Compare
This is a major issue for me, can I do anything to help getting this released? |
This package uses `dependencySatisfies` of `@embroider/macros` which is only reliable if a `peerDependency` for the package exists. If not, the macro checks for a dependency in the `ember-engines` package when a monorepo structure is used. This behaviour happens since embroider v0.50.1 which includes an important fix for monorepos: embroider-build/embroider#1070
c08f765
to
35fd62d
Compare
This package uses
dependencySatisfies
of@embroider/macros
which isonly reliable if a
peerDependency
for the package exists. If not, themacro checks for a dependency in the
ember-engines
package when amonorepo structure is used. This behaviour happens since embroider
v0.50.1 which includes an important fix for monorepos:
embroider-build/embroider#1070