Proposal: ember-cli-deprecation-workflow 3

[mixonic]

This issue proposes a major version release of ember-cli-deprecation-workflow: v3.0.0

The aims of a 3.0 are:

• [x] Swap from the vendor.js based implementation to one where the dependency must be manually added to app.js. https://github.com/mixonic/ember-cli-deprecation-workflow/pull/159
• [x] Update the oldest supported Ember version to be 3.28 (of course we will also confirm compatibility with Ember's bleeding edge) https://github.com/mixonic/ember-cli-deprecation-workflow/pull/159
• [x] Update the oldest supported Node version to be Node 16 https://github.com/mixonic/ember-cli-deprecation-workflow/pull/175
• [x] Adopt Volta edit: this is already done actually, but wasn't being honored in CI https://github.com/mixonic/ember-cli-deprecation-workflow/pull/170
• [x] Adopt ~pnpm~ npm https://github.com/mixonic/ember-cli-deprecation-workflow/pull/170
• [x] Migrate the repo home to the ember-cli org (or emberjs org?)
• [x] Ember CLI 4.12 or above
• [ ] All PRs and issues should use the label 3.0

Any other suggestions or concerns?

[mansona]

Hey thanks for this list 🎉 I have a few questions/comments about some of the specifics:

Update the oldest supported Node version to be Node 16

Is there any particular reason why we're not intending to make this Node 18 since v16 was EOL'd on 2023-09-11 ?

Adopt Volta

Do you have more info on the motivation behind this? We have recently removed volta from our test suite in Embroider because it was giving us so much grief. We found that the lockfile was good enough to pin the behaviour of pnpm locally and it also supports defining a version of Node directly in your .npmrc file

[mixonic]

Is there any particular reason why we're not intending to make this Node 18 since v16 was EOL'd on 2023-09-11 ?

Eh, we could. In general this kind of library is intended to help codebases on very old versions bridge the gap to newer versions. So I think some dragging of the feet for minimal version support is probably a good idea.

Do you have more info on the motivation behind this? We have recently https://github.com/embroider-build/embroider/pull/1594 because it was giving us so much grief. We found that the lockfile was good enough to pin the behaviour of pnpm locally and it also supports defining a version of Node directly in your .npmrc file

I'm not familiar with pnpm's pinning functionality at all. I thought Node's infra is what kept falling over with Volta.

Volta still lacks support for pnpm I just realized. Lame https://docs.volta.sh/advanced/pnpm

Yay that pnpm lockfile is supported by dependabot though! https://github.com/dependabot/dependabot-core/issues/1736

[simonihmig]

The first two checkboxes here can be checked I think!

One related minor cleanup PR: #181. With all broccoli dependencies removed, I think we could turn this addon into v2 format, couldn't we? As we don't have a dependency on ember-auto-import currently, this would be strictly speaking a breaking change (although probably nobody notice, unless they are light years behind), so something we should do before bumping the major!

Also this: https://github.com/mixonic/ember-cli-deprecation-workflow/issues/182

[mansona]

@simonihmig are we able to convert this to a v2 with the fact that we still need the vendor support to be able to catch deprecations in vendor code?