assert: fix loose deepEqual arrays with undefined and null failing by BridgeAR · Pull Request #61587 · nodejs/node · GitHub
Skip to content

assert: fix loose deepEqual arrays with undefined and null failing#61587

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
BridgeAR:BridgeAR/2026-01-30-fix-deep-equal
Feb 1, 2026
Merged

assert: fix loose deepEqual arrays with undefined and null failing#61587
nodejs-github-bot merged 1 commit intonodejs:mainfrom
BridgeAR:BridgeAR/2026-01-30-fix-deep-equal

Conversation

@BridgeAR
Copy link
Copy Markdown
Member

The comparison has to accept these as identical.

Fixes: #61583

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module. labels Jan 30, 2026
@BridgeAR BridgeAR marked this pull request as draft January 30, 2026 12:41
@BridgeAR BridgeAR force-pushed the BridgeAR/2026-01-30-fix-deep-equal branch from 3d8064f to 9d35d47 Compare January 30, 2026 12:43
The comparison has to accept these as identical.

Fixes: nodejs#61583
@BridgeAR BridgeAR force-pushed the BridgeAR/2026-01-30-fix-deep-equal branch from 9d35d47 to 225a1fd Compare January 30, 2026 12:50
@BridgeAR BridgeAR marked this pull request as ready for review January 30, 2026 12:50
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 30, 2026

@atlowChemi atlowChemi added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. request-ci Add this label to start a Jenkins CI on a PR. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jan 31, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 31, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@atlowChemi atlowChemi added the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 1, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 1, 2026
@nodejs-github-bot nodejs-github-bot merged commit 27a7bbc into nodejs:main Feb 1, 2026
95 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

aduh95 pushed a commit that referenced this pull request Feb 2, 2026
The comparison has to accept these as identical.

Fixes: #61583
PR-URL: #61587
Reviewed-By: Jithil P Ponnan <jithil@outlook.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
aduh95 pushed a commit that referenced this pull request Feb 2, 2026
The comparison has to accept these as identical.

Fixes: #61583
PR-URL: #61587
Reviewed-By: Jithil P Ponnan <jithil@outlook.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
aduh95 pushed a commit that referenced this pull request Feb 10, 2026
The comparison has to accept these as identical.

Fixes: #61583
PR-URL: #61587
Reviewed-By: Jithil P Ponnan <jithil@outlook.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
aduh95 pushed a commit that referenced this pull request Feb 14, 2026
The comparison has to accept these as identical.

Fixes: #61583
PR-URL: #61587
Reviewed-By: Jithil P Ponnan <jithil@outlook.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
aduh95 pushed a commit that referenced this pull request Feb 22, 2026
The comparison has to accept these as identical.

Fixes: #61583
PR-URL: #61587
Reviewed-By: Jithil P Ponnan <jithil@outlook.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

assert.deepEqual([null], [undefined]) behavior changed in v22.16.0 (undocumented breaking change)

8 participants