Skip to content

tools: remove custom logic for skipping test-strace-openat-openssl#62038

Merged
aduh95 merged 1 commit intonodejs:mainfrom
aduh95:test-strace-openat-openssl
Mar 2, 2026
Merged

tools: remove custom logic for skipping test-strace-openat-openssl#62038
aduh95 merged 1 commit intonodejs:mainfrom
aduh95:test-strace-openat-openssl

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Feb 28, 2026

Refs: #61987

@aduh95 aduh95 added dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. labels Feb 28, 2026
@aduh95 aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Feb 28, 2026
Copy link
Member

@gurgunday gurgunday left a comment

Choose a reason for hiding this comment

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

lgtm

@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 28, 2026
@aduh95 aduh95 force-pushed the test-strace-openat-openssl branch from 1384e59 to 97c6e3d Compare March 1, 2026 11:33
@nodejs-github-bot nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 2, 2026
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/62038
✔  Done loading data for nodejs/node/pull/62038
----------------------------------- PR info ------------------------------------
Title      tools: remove custom logic for skipping `test-strace-openat-openssl` (#62038)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     aduh95:test-strace-openat-openssl -> nodejs:main
Labels     author ready, dont-land-on-v20.x, dont-land-on-v22.x, dont-land-on-v24.x
Commits    1
 - tools: remove custom logic for skipping `test-strace-openat-openssl`
Committers 1
 - Antoine du Hamel <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/62038
Refs: https://github.com/nodejs/node/pull/61987
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gürgün Dayıoğlu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/62038
Refs: https://github.com/nodejs/node/pull/61987
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gürgün Dayıoğlu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last approving review:
   ⚠  - tools: remove custom logic for skipping `test-strace-openat-openssl`
   ℹ  This PR was created on Sat, 28 Feb 2026 08:08:55 GMT
   ✔  Approvals: 4
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/62038#pullrequestreview-3870462136
   ✔  - Colin Ihrig (@cjihrig): https://github.com/nodejs/node/pull/62038#pullrequestreview-3870523009
   ✔  - Gürgün Dayıoğlu (@gurgunday): https://github.com/nodejs/node/pull/62038#pullrequestreview-3870529022
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/62038#pullrequestreview-3870829468
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/22567442800

@aduh95 aduh95 merged commit 97c1312 into nodejs:main Mar 2, 2026
22 checks passed
@aduh95
Copy link
Contributor Author

aduh95 commented Mar 2, 2026

Landed in 97c1312

@aduh95 aduh95 deleted the test-strace-openat-openssl branch March 2, 2026 09:43
richardlau pushed a commit that referenced this pull request Mar 2, 2026
PR-URL: #62038
Refs: #61987
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gürgün Dayıoğlu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
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. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants