Skip to content

Conversation

mixonic
Copy link
Member

@mixonic mixonic commented May 28, 2021

Avoid accessing the Ember global which is deprecated per https://deprecations.emberjs.com/v3.x/#toc_ember-global

Discovered working on ember-cli/ember-cli-deprecation-workflow#118

@mixonic mixonic requested review from simonihmig and rwjblue May 28, 2021 11:28
@mixonic mixonic added the bug Something isn't working label May 28, 2021
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request May 28, 2021
* Avoid the Ember global in workflow runtime code.
* Ensure the test suite isn't using deprecated API by throwing unhandled
  deprecations. Address several uses of deprecated Ember API.
* Avoid reliance on ember-cli-babel < 7.26.6 which referenced the Ember
  global.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
Copy link
Contributor

@simonihmig simonihmig left a comment

Choose a reason for hiding this comment

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

Good catch, thank you!

The addon here needs some maintenance love, will try to move CI to Github asap, as Travis is not running at all anymore it seems 😕.

This is probably good to be merged anyway, but at least before releasing we should have a green CI...

@mixonic
Copy link
Member Author

mixonic commented May 28, 2021

I'll merge!

@mixonic mixonic merged commit d064744 into master May 28, 2021
@delete-merged-branch delete-merged-branch bot deleted the mixonic/no-global-ember branch May 28, 2021 17:46
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.
* Ensure the test suite isn't using deprecated API by throwing unhandled
  deprecations. Address several uses of deprecated Ember API.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.
* Ensure the test suite isn't using deprecated API by throwing unhandled
  deprecations. Address several uses of deprecated Ember API.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.
* Ensure the test suite isn't using deprecated API by throwing unhandled
  deprecations. Address several uses of deprecated Ember API.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.
* Ensure the test suite isn't using deprecated API by throwing unhandled
  deprecations. Address several uses of deprecated Ember API.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.
* Ensure the test suite isn't using deprecated API by throwing unhandled
  deprecations. Address several uses of deprecated Ember API.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.
* Bump ember-jquery to get
  emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 4, 2021
* Avoid the Ember global in workflow runtime code.

This patch is dependent on
#125 which
included the code at emberjs/ember-jquery#321
mixonic added a commit to ember-cli/ember-cli-deprecation-workflow that referenced this pull request Jun 12, 2021
* Avoid the Ember global in workflow runtime code.

This patch is dependent on
#125 which
included the code at emberjs/ember-jquery#321
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants