Skip to content

typings: fix typo on quic onSessionDatagram - #54064

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
1ilsang:fix/typings-typo
Jul 29, 2024
Merged

typings: fix typo on quic onSessionDatagram#54064
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
1ilsang:fix/typings-typo

Conversation

@1ilsang

Copy link
Copy Markdown
Member
image
  • Fix wrong typo

@H4adH4ad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you change your commit message to be more clear about what are you fixing?

Something like: typings: fix typo on quic onSessionDatagram

@1ilsang

Copy link
Copy Markdown
MemberAuthor

@H4ad

Hi, d2cf2df Done.

Thanks!

@anonriganonrig added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jul 27, 2024
@deokjinkimdeokjinkim changed the title typings: fix wrong typotypings: fix typo on quic onSessionDatagramJul 28, 2024
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 29, 2024
@nodejs-github-bot
nodejs-github-bot merged commit 5210cd8 into nodejs:mainJul 29, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 5210cd8

targos pushed a commit that referenced this pull request Jul 30, 2024
PR-URL: #54064
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Deokjin Kim <deokjin81.kim@gmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@RafaelGSSRafaelGSS mentioned this pull request Jul 30, 2024
RafaelGSS pushed a commit that referenced this pull request Aug 5, 2024
PR-URL: #54064
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Deokjin Kim <deokjin81.kim@gmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.typings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@1ilsang@nodejs-github-bot@lpinca@anonrig@deokjinkim@H4ad@jakecastelli@targos