Skip to content

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Apr 28, 2024

IIUC #52674 is not yet ready to land, so let's mark the test as flaky in the mean time.

Refs: #52630

@aduh95 aduh95 added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 28, 2024
@github-actions
Copy link
Contributor

Fast-track has been requested by @aduh95. Please 👍 to approve.

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Apr 28, 2024
@nodejs-github-bot
Copy link
Collaborator

@aduh95 aduh95 force-pushed the test-error-serdes-flaky branch from 17707a1 to 282d078 Compare April 28, 2024 20:52
@aduh95 aduh95 changed the title testL make test-error-serdes as flaky test: make test-error-serdes as flaky Apr 28, 2024
@nodejs-github-bot
Copy link
Collaborator

@aduh95 aduh95 merged commit 7c3dce0 into nodejs:main Apr 28, 2024
@aduh95
Copy link
Contributor Author

aduh95 commented Apr 28, 2024

Landed in 7c3dce0

@aduh95 aduh95 deleted the test-error-serdes-flaky branch April 28, 2024 23:10
aduh95 added a commit that referenced this pull request Apr 29, 2024
PR-URL: #52739
Refs: #52630
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
marco-ippolito pushed a commit that referenced this pull request May 2, 2024
PR-URL: #52739
Refs: #52630
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
marco-ippolito pushed a commit that referenced this pull request May 3, 2024
PR-URL: #52739
Refs: #52630
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
eliph4z pushed a commit to eliph4z/node that referenced this pull request Jun 20, 2024
PR-URL: nodejs#52739
Refs: nodejs#52630
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
bmeck pushed a commit to bmeck/node that referenced this pull request Jun 22, 2024
PR-URL: nodejs#52739
Refs: nodejs#52630
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track PRs that do not need to wait for 48 hours to land. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants