Skip to content

fix: use default import for express#28876

Merged
hybrist merged 1 commit into
angular:mainfrom
dgp1130:express-import
Nov 16, 2024
Merged

fix: use default import for express#28876
hybrist merged 1 commit into
angular:mainfrom
dgp1130:express-import

Conversation

@dgp1130

@dgp1130 dgp1130 commented Nov 15, 2024

Copy link
Copy Markdown
Collaborator

Now that we're using esModuleInterop, a default import is required.

@dgp1130 dgp1130 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Nov 15, 2024
@dgp1130
dgp1130 requested review from clydin and hybrist November 15, 2024 23:44
Now that we're using `esModuleInterop`, a default import is required.
@dgp1130 dgp1130 added target: rc This PR is targeted for the next release-candidate and removed target: patch This PR is targeted for the next patch release labels Nov 16, 2024
@dgp1130

dgp1130 commented Nov 16, 2024

Copy link
Copy Markdown
Collaborator Author

Targeting RC for now, this might make sense for v18 projects or earlier, but we can backport later as needed.

@dgp1130 dgp1130 added this to the v19 Candidates milestone Nov 16, 2024
@dgp1130 dgp1130 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 Nov 16, 2024
@dgp1130
dgp1130 removed the request for review from clydin November 16, 2024 01:22
@hybrist
hybrist merged commit 2e43ec6 into angular:main Nov 16, 2024
@hybrist

hybrist commented Nov 16, 2024

Copy link
Copy Markdown
Contributor

The changes were merged into the following branches: main, 19.0.x

@dgp1130
dgp1130 deleted the express-import branch November 18, 2024 18:31
@angular-automatic-lock-bot

Copy link
Copy Markdown

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 Dec 19, 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: @schematics/angular target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants