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.

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

doc: add CITGM to glossary.md #43316

Merged
merged 1 commit into from
Jun 5, 2022

Conversation

F3n67u
Copy link
Member

@F3n67u F3n67u commented Jun 5, 2022

I saw CITGM in some pull request comments, such as #42083 (comment), but I don't know what it means. We also have a need-citgm tag for pull request.

I think it makes sense to add it to our glossary to don't make new contributors confused.

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jun 5, 2022
@F3n67u F3n67u force-pushed the glossary-cigtm-v2 branch 2 times, most recently from 8a61ffe to 807c863 Compare June 5, 2022 06:58
Copy link
Contributor

@aduh95 aduh95 left a comment

Choose a reason for hiding this comment

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

Can you fix the commit message also please? (CIGTM -> CITGM)

glossary.md Outdated Show resolved Hide resolved
glossary.md Outdated Show resolved Hide resolved
@F3n67u F3n67u changed the title doc: add CIGTM to glossary.md doc: add CITGM to glossary.md Jun 5, 2022
@F3n67u
Copy link
Member Author

F3n67u commented Jun 5, 2022

Can you fix the commit message also please? (CIGTM -> CITGM)

Done, thanks. PTAL.

@F3n67u F3n67u requested a review from aduh95 June 5, 2022 09:05
@aduh95 aduh95 added fast-track PRs that do not need to wait for 48 hours to land. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Jun 5, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2022

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

@marsonya marsonya added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 5, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 5, 2022
@nodejs-github-bot nodejs-github-bot merged commit 815e74b into nodejs:master Jun 5, 2022
@nodejs-github-bot
Copy link
Collaborator

Landed in 815e74b

italojs pushed a commit to italojs/node that referenced this pull request Jun 6, 2022
PR-URL: nodejs#43316
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
danielleadams pushed a commit that referenced this pull request Jun 11, 2022
PR-URL: #43316
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
@danielleadams danielleadams mentioned this pull request Jun 11, 2022
danielleadams pushed a commit that referenced this pull request Jun 13, 2022
PR-URL: #43316
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
danielleadams pushed a commit that referenced this pull request Jun 13, 2022
PR-URL: #43316
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
targos pushed a commit that referenced this pull request Jul 12, 2022
PR-URL: #43316
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
targos pushed a commit that referenced this pull request Jul 31, 2022
PR-URL: #43316
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
guangwong pushed a commit to noslate-project/node that referenced this pull request Oct 10, 2022
PR-URL: nodejs/node#43316
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants