Ember.js: [CLEANUP] remove enabled feature flags on release

[chancancode]

[mul53]

@chancancode can i have a crack at this?

[rajankumar549]

Hey, can i help to fix this issue. I want to contribute in this project. @chancancode

[chancancode]

@mul53 @rajankumar549 you don't need permission, just go ahead and open PR. Preferably one feature flag at a time and maybe add a note here on what you are working on so others can pick something else.

[neonick]

@chancancode seems it could be closed?

[kanavpruthi]

I am a beginner, and id like to work on a new issue here

[chancancode]

@kanavpruthi the specific set of enabled features to remove has changed since this was opened, but there are now new ones that needed to be cleaned up https://github.com/emberjs/ember.js/blob/master/packages/@ember/canary-features/index.ts

[Alpine-lines]

Should all of the features in /packages/@ember/canary-features/index.ts have their respective feature flags removed?

[pzuraq]

@Alpine-lines any flag that has been set to true, and has been released in a minor version of Ember, should be removed. Currently, these ones: https://github.com/emberjs/ember.js/blob/master/packages/@ember/canary-features/index.ts#L19-L23

[pzuraq]

The best way to approach this is with one PR per feature flag, to make it easier to review quickly