Skip to content

partially address ember barrel depprecation via some light dependency updating #316

Merged
evoactivity merged 5 commits intomasterfrom
nvp/globals-deprecation
Apr 17, 2026
Merged

partially address ember barrel depprecation via some light dependency updating #316
evoactivity merged 5 commits intomasterfrom
nvp/globals-deprecation

Conversation

@NullVoxPopuli
Copy link
Copy Markdown
Contributor

@NullVoxPopuli NullVoxPopuli commented Apr 17, 2026

I can't fix all of this in this PR, beecause I have to remove fastboot.

I can't remove fastboot without a docs solution (vite ember-ssr), and I can't use vite-ember-ssr until the pproject is on vite.

So after this PR, I'll update both apps to vite, and add SSR/SSG where approp

@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review April 17, 2026 12:05
Comment thread docs/tests/test-helper.js
import config from 'docs/config/environment';
import { setApplication } from '@ember/test-helpers';
import { start } from 'ember-qunit';
import { loadTests } from 'ember-qunit/test-loader';
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this should all be done via e-c-u
this is just part of the upgrade and is very selective

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

everyone says ECU PRs are too big to review.

and ECU doesn't do anything in a targeted way -- it's a sledge hammer.
It also tends to choose out of date things.
It's not the best tool for how I like doing upgrades.

Copy link
Copy Markdown
Contributor

@evoactivity evoactivity left a comment

Choose a reason for hiding this comment

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

PR title doesn't match the changes made.

@NullVoxPopuli NullVoxPopuli changed the title Fix ember barrel depprecation partially address ember barrel depprecation via some light dependency updating Apr 17, 2026
@evoactivity evoactivity merged commit 80832b1 into master Apr 17, 2026
16 of 18 checks passed
@github-actions github-actions bot mentioned this pull request Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants