Skip to content
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 #1872

Merged
merged 30 commits into from
Apr 10, 2024
Merged

Merge stable #1872

merged 30 commits into from
Apr 10, 2024

Conversation

mansona
Copy link
Member

@mansona mansona commented Apr 10, 2024

No description provided.

bertdeblock and others added 30 commits March 4, 2024 11:45
this was causing some issues for us in the vite work because we now have a real babel config. This could have caused a bunch of other bugs that this really shouldn't be using
make sure @embroider/macros doesn't try to load a babel config
test updates to match new expectations
…built-ins

resolver transform to emit imports for helper and modifiers that need…
… hbs files instead of js that don't exist in their codebase
add a namespace option for public-assets plugin
release-plan is supposed to report unlabelled PRs in the Prepare Release pr but it hasn't been working correctly. I figured out that the issue is that the pull request action won't open a PR unless there are some file changes and `stderr.log` is covered by our .gitignore file.

This PR fixes that and it also makes sure that the stderr.txt doesn't make it into the PR when the release-plan job is successful.
fix release-plan unlabelled changes PR
# Conflicts:
#	packages/addon-dev/src/rollup-public-assets.ts
#	packages/shared-internals/src/index.ts
#	pnpm-lock.yaml
@mansona mansona merged commit 80f442d into main Apr 10, 2024
92 checks passed
@mansona mansona deleted the merge-stable branch April 10, 2024 15:54
@github-actions github-actions bot mentioned this pull request Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants