-
-
Notifications
You must be signed in to change notification settings - Fork 109
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 checking for packaged/dev modes #1698
Fix checking for packaged/dev modes #1698
Conversation
In the process of removing `electron-is-dev` the way that we determine if we're in development environments changed, and as a result we need to invert the conditional. Fixes: adopted-ember-addons#1697
@jagthedrummer have you confirmed that this does indeed work? |
@RobbieTheWagner in my pre-existing app making this change to In a new The output in the console looks like this:
Kinda seems like two different, but closely related issues. But I don't really know. |
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.
LGTM -- thank you!
e2e1c29
into
adopted-ember-addons:main
Ugh, the build failure...I'll open a PR in a sec updating our Node version in CI which should address that. |
In the process of removing
electron-is-dev
(#1640) the way that we determine if we're in development environments changed, and as a result we need to invert the conditional.Fixes: #1697