-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Release: Patch 8.4.6 #29679
Release: Patch 8.4.6 #29679
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
5 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile
773f9b8
to
151e413
Compare
74c24bd
to
4a81359
Compare
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 15e14c4. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 2 targetsSent with 💌 from NxCloud. |
7def712
to
4b31c87
Compare
Very important patch for angular 19, we are blocked by that issue, would love to get it as soon as possible |
For sure! I'm working hard to get this in. There are unrelated changes which are blocking our CI but they will be fixed shortly! |
Angular: Default to standalone components in Angular v19 (cherry picked from commit 8ba4fb5)
Docs: Maintenance - Fix snippets and documentation (cherry picked from commit 896be2b)
Portable stories: Support multiple annotation notations from preview (cherry picked from commit 020074b)
Build: Fix portable stories tests by moving from jsdom to happy-dom (cherry picked from commit 0a656a7)
16447d0
to
e243c64
Compare
…windows Addon Test: Use pathe for better windows support (cherry picked from commit 1bb5920)
Frameworks: Add Vite 6 support (cherry picked from commit 8e5be14)
…plugin-5 Svelte: Support `@sveltejs/vite-plugin-svelte` v5 (cherry picked from commit e870c8c)
Build: Attempt to fix Vite 6 sandboxes (cherry picked from commit a4ae081)
## Third-party deps support for Angular v19 - [x] `jest-preset-angular` - [x] PRs: - [x] thymikee/jest-preset-angular#2835 - [x] Released: - [x] RC: https://github.com/thymikee/jest-preset-angular/releases/tag/v14.4.0-rc.0 - [x] Stable: https://github.com/thymikee/jest-preset-angular/releases/tag/v14.4.0 - [x] Angular ESLint - [x] PRs: - [x] angular-eslint/angular-eslint#2109 - [x] Released: - [x] https://github.com/angular-eslint/angular-eslint/releases/tag/v19.0.0 - [x] Storybook - [x] PRs: - [x] storybookjs/storybook#29659 - [x] storybookjs/storybook#29677 - [x] Released: - [x] storybookjs/storybook#29679 - [ ] NgRx - [x] PRs: - [x] ngrx/platform#4602 - [ ] Released: - [x] Beta: https://github.com/ngrx/platform/blob/main/CHANGELOG.md#1900-beta0-2024-11-20 - [ ] Stable: - [ ] Analog - [x] PRs: - [x] analogjs/analog#1447 - [x] analogjs/analog#1451 - [ ] Released: - [x] Beta: https://github.com/analogjs/analog/releases/tag/v1.10.0-beta.6 - [ ] Stable: <!-- Please make sure you have read the submission guidelines before posting an PR --> <!-- https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr --> <!-- Please make sure that your commit message follows our format --> <!-- Example: `fix(nx): must begin with lowercase` --> <!-- If this is a particularly complex change or feature addition, you can request a dedicated Nx release for this pull request branch. Mention someone from the Nx team or the `@nrwl/nx-pipelines-reviewers` and they will confirm if the PR warrants its own release for testing purposes, and generate it for you if appropriate. --> ## Current Behavior <!-- This is the behavior we have today --> Angular v19 is not supported. ## Expected Behavior <!-- This is the behavior we should expect with the changes in this PR --> Angular v19 should be supported. ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes #29028
This is an automated pull request that bumps the version from
8.4.5
to8.4.6
.Once this pull request is merged, it will trigger a new release of version
8.4.6
.If you're not a core maintainer with permissions to release you can ignore this pull request.
To do
Before merging the PR, there are a few QA steps to go through:
And for each change below:
This is a list of all the PRs merged and commits pushed directly to
next
, that will be part of this release:@sveltejs/vite-plugin-svelte
v5 #29731🍒 Manual cherry picking needed!
The following pull requests could not be cherry-picked automatically because it resulted in merge conflicts.
For each pull request below, you need to either manually cherry pick it, or discard it by replacing the "patch:yes" label with "patch:no" on the PR and re-generate this PR.
git cherry-pick -m1 -x 1bb5920115f5a4978bbe58bd51a10e1591cf5c06
git cherry-pick -m1 -x 8e5be14c49a2b42ee473b79cfd6b78858ca2e172
git cherry-pick -m1 -x e870c8c7a390520aba5d27bce7dcf3551a53bf49
git cherry-pick -m1 -x a4ae081b261f9d1f274185a9478bc48f930ea668
If you've made any changes doing the above QA (change PR titles, revert PRs), manually trigger a re-generation of this PR with this workflow and wait for it to finish. It will wipe your progress in this to do, which is expected.
Feel free to manually commit any changes necessary to this branch after you've done the last re-generation, following the Make Manual Changes section in the docs, especially if you're making changes to the changelog.
When everything above is done:
Generated changelog
8.4.6
@sveltejs/vite-plugin-svelte
v5 - #29731, thanks @JReinhold!