Skip to content

replace hasOwnProperty with Object.hasOwn #1329

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

Merged
merged 3 commits into from
Jun 3, 2025

Conversation

bjohansebas
Copy link
Member

This change is a breaking change for Node <16.9

Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
@bjohansebas bjohansebas force-pushed the bjohasenasb/hasOwn branch from f3981b3 to 245b692 Compare June 2, 2025 18:33
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
@bjohansebas bjohansebas force-pushed the bjohasenasb/hasOwn branch from 245b692 to 59ba2c4 Compare June 2, 2025 18:34
@bjohansebas bjohansebas mentioned this pull request Jun 2, 2025
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
@UlisesGascon
Copy link
Member

As the minimum supported version will be 18, based on #1303. Then we can include this 💪

@UlisesGascon UlisesGascon self-assigned this Jun 3, 2025
@UlisesGascon UlisesGascon merged commit 690fc51 into expressjs:v3 Jun 3, 2025
29 of 38 checks passed
@bjohansebas bjohansebas deleted the bjohasenasb/hasOwn branch June 3, 2025 17:46
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.

2 participants