Skip to content

Commit

Permalink
Remove computedPolyfill
Browse files Browse the repository at this point in the history
According to @rwjblue and @teddyzeenny we should no longer need this. Fixes emberjs#758
  • Loading branch information
RobbieTheWagner authored and cyril-sf committed Mar 30, 2022
1 parent e0b0013 commit af6e451
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 55 deletions.
36 changes: 0 additions & 36 deletions ember_debug/addons/ember-new-computed/index.js

This file was deleted.

15 changes: 0 additions & 15 deletions ember_debug/addons/ember-new-computed/utils/can-use-new-syntax.js

This file was deleted.

3 changes: 1 addition & 2 deletions ember_debug/models/promise.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import computedPolyfill from '../addons/ember-new-computed/index';
const Ember = window.Ember;
const { typeOf, Object: EmberObject, computed, A } = Ember;

const dateComputed = function() {
return computedPolyfill({
return computed({
get() {
return null;
},
Expand Down
3 changes: 1 addition & 2 deletions ember_debug/promise-debug.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import PortMixin from 'ember-debug/mixins/port-mixin';
import PromiseAssembler from 'ember-debug/libs/promise-assembler';
import computedPolyfill from './addons/ember-new-computed/index';
const Ember = window.Ember;
const { computed, Object: EmberObject, RSVP, A, run } = Ember;
const { oneWay } = computed;
Expand Down Expand Up @@ -80,7 +79,7 @@ export default EmberObject.extend(PortMixin, {
}
},

instrumentWithStack: computedPolyfill({
instrumentWithStack: computed({
get() {
return !!this.get('session').getItem('promise:stack');
},
Expand Down

0 comments on commit af6e451

Please sign in to comment.