Skip to content

src: fix some typo errors and rename some variables#64301

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
Archkon:typo
Jul 12, 2026
Merged

src: fix some typo errors and rename some variables#64301
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
Archkon:typo

Conversation

@Archkon

@Archkon Archkon commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Just some comment changes and rename variables for better understanding

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/quic
  • @nodejs/sqlite

@nodejs-github-bot nodejs-github-bot added the needs-ci PRs that need a full CI run. label Jul 5, 2026
Comment thread test/parallel/test-repl-colors.js Outdated
Comment thread test/sequential/test-net-response-size.js Outdated
Comment thread test/sequential/test-watch-mode.mjs Outdated
@nodejs-github-bot

This comment was marked as outdated.

@Archkon

Archkon commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

@jasnell
I can't access jenkins ci so could you review what happend for these failed tests when available? Thanks!

@jasnell

jasnell commented Jul 7, 2026

Copy link
Copy Markdown
Member

CI failures are flakes. Running it again.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Signed-off-by: Archkon <180910180+Archkon@users.noreply.github.com>
@Archkon

Archkon commented Jul 11, 2026

Copy link
Copy Markdown
Contributor Author

@jasnell I think this pr could land on main branch and it open for one week so could you add it to commit-queue? Or I need to do anything else?

@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 11, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 11, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@Archkon

Archkon commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

@lpinca Actually, this PR had passed the Jenkins CI tests before, but I unintentionally force-pushed the branch while handling several local branches in batch. Anyway, thank you for triggering Jenkins CI again.Could you please add this PR to the commit queue? Or sorry to bother you to took a look at this trivial pr that could be fast-track if you are avilable:#64400

@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 12, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 12, 2026
@nodejs-github-bot nodejs-github-bot merged commit 7c53fb5 into nodejs:main Jul 12, 2026
71 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 7c53fb5

@Archkon Archkon deleted the typo branch July 12, 2026 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants