Skip to content

Fix perf benchmark: add import.meta.env?.DEV replacement to benchmark app vite config#21117

Closed
Copilot wants to merge 3 commits intonvp/no-more-macrosfrom
copilot/sub-pr-21115
Closed

Fix perf benchmark: add import.meta.env?.DEV replacement to benchmark app vite config#21117
Copilot wants to merge 3 commits intonvp/no-more-macrosfrom
copilot/sub-pr-21115

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

  • Reverted all changes: smoke-tests/benchmark-app/vite.config.mjs and packages/@ember/-internals/metal/lib/decorator.ts restored to original state

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…d re-apply COMPUTED_GETTERS fix

Co-authored-by: NullVoxPopuli <199018+NullVoxPopuli@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove macros in favor of import.meta.env?.DEV Fix perf benchmark: add import.meta.env?.DEV replacement to benchmark app vite config Feb 25, 2026
@NullVoxPopuli
Copy link
Contributor

@copilot a type error was not the problem. do not change the vite config, and undo all the changes so far in this branch.

Co-authored-by: NullVoxPopuli <199018+NullVoxPopuli@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 26, 2026

@copilot a type error was not the problem. do not change the vite config, and undo all the changes so far in this branch.

Done. All changes have been reverted in 2cf79a3 — both smoke-tests/benchmark-app/vite.config.mjs and packages/@ember/-internals/metal/lib/decorator.ts are back to their original state.

@NullVoxPopuli NullVoxPopuli deleted the copilot/sub-pr-21115 branch February 26, 2026 00:16
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.

2 participants