test: add fast api tests for getLibuvNow() by anonrig · Pull Request #58022 · nodejs/node · GitHub
Skip to content

test: add fast api tests for getLibuvNow()#58022

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
anonrig:yagiz/add-libuv-fast-api
Apr 27, 2025
Merged

test: add fast api tests for getLibuvNow()#58022
nodejs-github-bot merged 1 commit intonodejs:mainfrom
anonrig:yagiz/add-libuv-fast-api

Conversation

@anonrig
Copy link
Copy Markdown
Member

@anonrig anonrig commented Apr 25, 2025

Adds a test for the v8 fast api we use for getLibuvNow() method.

@anonrig anonrig requested review from jasnell and targos April 25, 2025 14:25
@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. timers Issues and PRs related to the timers subsystem / setImmediate, setInterval, setTimeout. labels Apr 25, 2025
@anonrig anonrig force-pushed the yagiz/add-libuv-fast-api branch from 1171c3a to ee7237f Compare April 25, 2025 14:25
@anonrig anonrig changed the title Yagiz/add libuv fast api test: add fast api tests for getLibuvNow() Apr 25, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 25, 2025 •
edited
Loading

@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Apr 25, 2025
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 25, 2025
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

nodejs-github-bot commented Apr 25, 2025

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

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

nodejs-github-bot commented Apr 25, 2025

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

@anonrig anonrig force-pushed the yagiz/add-libuv-fast-api branch from ee7237f to 5b01f13 Compare April 26, 2025 21:02
@anonrig anonrig requested a review from H4ad April 26, 2025 22:48
@anonrig anonrig added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 27, 2025
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 27, 2025
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

nodejs-github-bot commented Apr 27, 2025

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

@anonrig anonrig added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 27, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 27, 2025
@nodejs-github-bot nodejs-github-bot merged commit 145c6a2 into nodejs:main Apr 27, 2025
60 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

nodejs-github-bot commented Apr 27, 2025

RafaelGSS pushed a commit that referenced this pull request May 1, 2025
@anonrig @RafaelGSS
test: add fast api tests for getLibuvNow()
0c027cd 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
@RafaelGSS RafaelGSS mentioned this pull request May 1, 2025
Merged
RafaelGSS pushed a commit that referenced this pull request May 2, 2025
@anonrig @RafaelGSS
test: add fast api tests for getLibuvNow()
3d34c5f 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
aduh95 pushed a commit that referenced this pull request May 6, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
ebce0f8 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
aduh95 pushed a commit that referenced this pull request May 6, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
31e202c 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
RafaelGSS pushed a commit that referenced this pull request May 14, 2025
@anonrig @RafaelGSS
test: add fast api tests for getLibuvNow()
ec66174 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
aduh95 pushed a commit that referenced this pull request May 16, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
558cf75 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
aduh95 pushed a commit that referenced this pull request May 17, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
ac48aad 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
aduh95 pushed a commit that referenced this pull request May 17, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
1871497 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
aduh95 pushed a commit that referenced this pull request May 17, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
4ae425d 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
aduh95 pushed a commit that referenced this pull request May 18, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
a31f538 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
@github-actions github-actions Bot mentioned this pull request May 19, 2025
Merged
aduh95 pushed a commit that referenced this pull request May 19, 2025
@anonrig @aduh95
test: add fast api tests for getLibuvNow()
b407692 
PR-URL: #58022
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
@ghost ghost mentioned this pull request Jun 8, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@H4ad H4ad H4ad approved these changes

@aduh95 aduh95 aduh95 approved these changes

@targos targos Awaiting requested review from targos

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. timers Issues and PRs related to the timers subsystem / setImmediate, setInterval, setTimeout.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@anonrig @nodejs-github-bot @jasnell @H4ad @aduh95