Skip to content
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.

Sign up for GitHub

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

Jump to bottom

test: skip v8-updates/test-linux-perf-logger #52821

Merged
merged 1 commit into from
May 5, 2024
Merged

test: skip v8-updates/test-linux-perf-logger #52821

merged 1 commit into from
May 5, 2024

Conversation

targos
Copy link
Member

@targos targos commented May 3, 2024 •
edited
Loading

@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 May 3, 2024
@nodejs-github-bot
Copy link
Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=benchmark-ubuntu2204-intel-64,v8test=v8test/5956/

@nodejs-github-bot
Copy link
Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-s390x,v8test=v8test/5956/

@nodejs-github-bot
Copy link
Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-ppc64le,v8test=v8test/5956/

@nodejs-github-bot

This comment was marked as duplicate.

Sign in to view
@nodejs-github-bot
Copy link
Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=benchmark-ubuntu2204-intel-64,v8test=v8test/5958/

@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label May 4, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 4, 2024
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/58913/

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/58930/

@targos targos added the commit-queue Add this label to land a pull request using GitHub Actions. label May 5, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 5, 2024
@nodejs-github-bot nodejs-github-bot merged commit 54f1e0a into nodejs:main May 5, 2024
57 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 54f1e0a

@targos targos deleted the targos-patch-1 branch May 5, 2024 12:27
@targos targos mentioned this pull request May 6, 2024
Open
lukealbao added a commit to lukealbao/node that referenced this pull request May 7, 2024
@lukealbao
test: unskip v8-updates/test-linux-perf-logger
cdc89d0 
Refs: nodejs#51308 nodejs#52821
@lukealbao lukealbao mentioned this pull request May 7, 2024
Closed
Ch3nYuY pushed a commit to Ch3nYuY/node that referenced this pull request May 8, 2024
@targos
test: skip v8-updates/test-linux-perf-logger
40d385b 
Refs: nodejs#51308
PR-URL: nodejs#52821
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
targos added a commit that referenced this pull request May 8, 2024
@targos
test: skip v8-updates/test-linux-perf-logger
1448959 
Refs: #51308
PR-URL: #52821
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@targos targos mentioned this pull request May 13, 2024
Merged
richardlau pushed a commit that referenced this pull request May 29, 2024
@lukealbao @richardlau
test: unskip v8-updates/test-linux-perf-logger
f671dcb 
Refs: #51308
Refs: #52821
PR-URL: #52869
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
targos pushed a commit that referenced this pull request Jun 1, 2024
@lukealbao @targos
test: unskip v8-updates/test-linux-perf-logger
c1138db 
Refs: #51308
Refs: #52821
PR-URL: #52869
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
marco-ippolito pushed a commit that referenced this pull request Jun 17, 2024
@targos @marco-ippolito
test: skip v8-updates/test-linux-perf-logger
0addeb2 
Refs: #51308
PR-URL: #52821
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@marco-ippolito marco-ippolito mentioned this pull request Jun 17, 2024
Merged
EliphazBouye pushed a commit to EliphazBouye/node that referenced this pull request Jun 20, 2024
@targos @EliphazBouye
test: skip v8-updates/test-linux-perf-logger
340f434 
Refs: nodejs#51308
PR-URL: nodejs#52821
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
EliphazBouye pushed a commit to EliphazBouye/node that referenced this pull request Jun 20, 2024
@lukealbao @EliphazBouye
test: unskip v8-updates/test-linux-perf-logger
5a80813 
Refs: nodejs#51308
Refs: nodejs#52821
PR-URL: nodejs#52869
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
bmeck pushed a commit to bmeck/node that referenced this pull request Jun 22, 2024
@targos @bmeck
test: skip v8-updates/test-linux-perf-logger
b12bab0 
Refs: nodejs#51308
PR-URL: nodejs#52821
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
bmeck pushed a commit to bmeck/node that referenced this pull request Jun 22, 2024
@lukealbao @bmeck
test: unskip v8-updates/test-linux-perf-logger
aa5f3e5 
Refs: nodejs#51308
Refs: nodejs#52821
PR-URL: nodejs#52869
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daeyeon daeyeon daeyeon approved these changes

@richardlau richardlau richardlau approved these changes

@lpinca lpinca lpinca approved these changes

@jasnell jasnell jasnell approved these changes

Assignees
No one assigned
Labels
needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@targos @nodejs-github-bot @jasnell @lpinca @richardlau @daeyeon