Skip to content

Conversation

@alan-agius4
Copy link
Collaborator

BREAKING CHANGE: Node.js support for versions <18.19.1 and <20.11.1 has been removed.

@alan-agius4 alan-agius4 added the target: major This PR is targeted for the next major release label Mar 11, 2024
@angular-robot angular-robot bot added detected: breaking change PR contains a commit with a breaking change area: build & ci Related the build and CI infrastructure of the project labels Mar 11, 2024
@alan-agius4 alan-agius4 force-pushed the node-js-18 branch 6 times, most recently from 3df5baa to e525296 Compare March 11, 2024 12:14
(harness, setupTarget, isViteRun) => {
// TODO(fix-vite): currently this is broken in vite.
(isViteRun ? xdescribe : describe)('option: "allowedHosts"', () => {
(isViteRun ? xdescribe : xdescribe)('option: "allowedHosts"', () => {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These tests just correctly return the page instead of failing.

https://github.com/angular/angular-cli/actions/runs/8232140697/job/22508914513?pr=27249

@alan-agius4 alan-agius4 added this to the v18 Candidates milestone Mar 11, 2024
@alan-agius4 alan-agius4 requested a review from clydin March 11, 2024 12:46
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 11, 2024
@alan-agius4 alan-agius4 marked this pull request as ready for review March 11, 2024 12:46
Update ng-dev setup to work with 18.19 For more details, refer to: TypeStrong/ts-node#2094
@alan-agius4 alan-agius4 force-pushed the node-js-18 branch 3 times, most recently from eea9ac8 to 49958a8 Compare March 12, 2024 14:27
…and <20.11.1

BREAKING CHANGE: Node.js support for versions <18.19.1 and <20.11.1 has been removed.
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 12, 2024
@alan-agius4 alan-agius4 merged commit c7b2085 into angular:main Mar 12, 2024
@alan-agius4 alan-agius4 deleted the node-js-18 branch March 12, 2024 15:51
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project detected: breaking change PR contains a commit with a breaking change target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants