-
Notifications
You must be signed in to change notification settings - Fork 142
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge stable into main #1996
Merge stable into main #1996
Conversation
… consumer app & remove ember-cli-eslint
…able Update `fast-sourcemap-concat` to remove dependency `lodash.template` for consumer apps (security vulnerability)
Prepare Release
It's a bug that most of the MacrosConfig state is synchronized between copies of @embroider/macros but the globalConfigs object is not.
update node to latest LTS for CI
MacrosConfig should sync globalConfigs between copies
Prepare Release
# Conflicts: # .release-plan.json # packages/core/package.json # packages/macros/src/macros-config.ts # pnpm-lock.yaml
Update typescript and fix issues with Typescript 5.5
# Conflicts: # package.json # pnpm-lock.yaml
looks like you also need the windows fix https://github.com/embroider-build/embroider/pull/1855/files#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03f |
@patricklx any idea why we might be hitting this now? what changed? |
I've added @patricklx 's change because I tested this locally and wasn't able to re-create the issue 🫠 If this fixes CI we can just merge it and move on, Windows shouldn't be blocking our process like this 😭 |
No description provided.