Skip to content

doc: amend support tier qualifier - #42805

Closed
gireeshpunathil wants to merge 1 commit into
nodejs:mainfrom
gireeshpunathil:support-tiers-tier3-define
Closed

doc: amend support tier qualifier#42805
gireeshpunathil wants to merge 1 commit into
nodejs:mainfrom
gireeshpunathil:support-tiers-tier3-define

Conversation

@gireeshpunathil

Copy link
Copy Markdown
Member

Amend Tier 3 category definition to remove a
tentative clause and introduce a more concrete one.

Refs: #42802

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Apr 21, 2022

@RafaelGSSRafaelGSS 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.

LGTM.

Comment threaddoc/contributing/diagnostic-tooling-support-tiers.md Outdated
Comment threaddoc/contributing/diagnostic-tooling-support-tiers.md Outdated
@gireeshpunathil

Copy link
Copy Markdown
MemberAuthor

this can have more reviews.

@mcollinamcollina 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.

lgtm

@gireeshpunathilgireeshpunathil added the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 12, 2023
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 12, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/42805
✔ Done loading data for nodejs/node/pull/42805
----------------------------------- PR info ------------------------------------
Title doc: amend support tier qualifier (#42805)
Author Gireesh Punathil (@gireeshpunathil)
Branch gireeshpunathil:support-tiers-tier3-define -> nodejs:main
Labels doc
Commits 2
- doc: amend support tier qualifier
- fixup: reword based on review comments
Committers 1
- Gireesh Punathil PR-URL: https://github.com/nodejs/node/pull/42805
Refs: https://github.com/nodejs/node/pull/42802
Reviewed-By: Rafael Gonzaga Reviewed-By: Matteo Collina Reviewed-By: Darshan Sen ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/42805
Refs: https://github.com/nodejs/node/pull/42802
Reviewed-By: Rafael Gonzaga Reviewed-By: Matteo Collina Reviewed-By: Darshan Sen --------------------------------------------------------------------------------
ℹ This PR was created on Thu, 21 Apr 2022 04:46:33 GMT
✔ Approvals: 3
✔ - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/42805#pullrequestreview-948787072
✔ - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/42805#pullrequestreview-1336000383
✔ - Darshan Sen (@RaisinTen) (TSC): https://github.com/nodejs/node/pull/42805#pullrequestreview-1336014453
✔ Last GitHub CI successful
ℹ Green GitHub CI is sufficient
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 42805
From https://github.com/nodejs/node
* branch refs/pull/42805/merge -> FETCH_HEAD
✔ Fetched commits as 7f2ab4e62985..e4da26ed9005
--------------------------------------------------------------------------------
[main 7e2d9f14fd] doc: amend support tier qualifier
Author: Gireesh Punathil Date: Thu Apr 21 10:11:32 2022 +0530
1 file changed, 4 insertions(+), 3 deletions(-)
[main 6ffb3148ac] fixup: reword based on review comments
Author: Gireesh Punathil Date: Sat Mar 11 19:01:42 2023 +0530
1 file changed, 3 insertions(+), 4 deletions(-)
✔ Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)

Executing: git node land --amend --yes
⚠ Found Refs: #42802, skipping..
--------------------------------- New Message ----------------------------------
doc: amend support tier qualifier

Amend Tier 3 category definition to remove a
tentative clause and introduce a more concrete one.

Refs: #42802
PR-URL: #42805
Reviewed-By: Rafael Gonzaga rafael.nunu@hotmail.com
Reviewed-By: Matteo Collina matteo.collina@gmail.com
Reviewed-By: Darshan Sen raisinten@gmail.com

[detached HEAD 8376941e12] doc: amend support tier qualifier
Author: Gireesh Punathil gpunathi@in.ibm.com
Date: Thu Apr 21 10:11:32 2022 +0530
1 file changed, 4 insertions(+), 3 deletions(-)
Rebasing (3/4)
Rebasing (4/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
fixup: reword based on review comments

PR-URL: #42805
Refs: #42802
Reviewed-By: Rafael Gonzaga rafael.nunu@hotmail.com
Reviewed-By: Matteo Collina matteo.collina@gmail.com
Reviewed-By: Darshan Sen raisinten@gmail.com

[detached HEAD 1952bf7a4c] fixup: reword based on review comments
Author: Gireesh Punathil gpunathi@in.ibm.com
Date: Sat Mar 11 19:01:42 2023 +0530
1 file changed, 3 insertions(+), 4 deletions(-)

Successfully rebased and updated refs/heads/main.

ℹ Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/4397476877

@gireeshpunathilgireeshpunathil added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Mar 12, 2023
Amend Tier 3 category definition to remove a
tentative clause and introduce a more concrete one.
Refs: nodejs#42802
@gireeshpunathil
gireeshpunathilforce-pushed the support-tiers-tier3-define branch from e4da26e to d5b971fCompareMarch 12, 2023 14:30
@gireeshpunathilgireeshpunathil added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. labels Mar 12, 2023
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 12, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/42805
✔ Done loading data for nodejs/node/pull/42805
----------------------------------- PR info ------------------------------------
Title doc: amend support tier qualifier (#42805)
Author Gireesh Punathil (@gireeshpunathil)
Branch gireeshpunathil:support-tiers-tier3-define -> nodejs:main
Labels doc
Commits 1
- doc: amend support tier qualifier
Committers 1
- Gireesh Punathil PR-URL: https://github.com/nodejs/node/pull/42805
Refs: https://github.com/nodejs/node/pull/42802
Reviewed-By: Rafael Gonzaga Reviewed-By: Matteo Collina Reviewed-By: Darshan Sen ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/42805
Refs: https://github.com/nodejs/node/pull/42802
Reviewed-By: Rafael Gonzaga Reviewed-By: Matteo Collina Reviewed-By: Darshan Sen --------------------------------------------------------------------------------
⚠ Commits were pushed since the last review:
⚠ - doc: amend support tier qualifier
ℹ This PR was created on Thu, 21 Apr 2022 04:46:33 GMT
✔ Approvals: 3
✔ - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/42805#pullrequestreview-948787072
✔ - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/42805#pullrequestreview-1336000383
✔ - Darshan Sen (@RaisinTen) (TSC): https://github.com/nodejs/node/pull/42805#pullrequestreview-1336014453
✔ Last GitHub CI successful
ℹ Green GitHub CI is sufficient
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/4398393764

gireeshpunathil added a commit that referenced this pull request Mar 13, 2023
Amend Tier 3 category definition to remove a
tentative clause and introduce a more concrete one.
Refs: #42802
PR-URL: #42805
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
@gireeshpunathil

Copy link
Copy Markdown
MemberAuthor

landed in 59dd9ca

targos pushed a commit that referenced this pull request Mar 13, 2023
Amend Tier 3 category definition to remove a
tentative clause and introduce a more concrete one.
Refs: #42802
PR-URL: #42805
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
targos pushed a commit that referenced this pull request Mar 14, 2023
Amend Tier 3 category definition to remove a
tentative clause and introduce a more concrete one.
Refs: #42802
PR-URL: #42805
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
@targostargos mentioned this pull request Mar 14, 2023
danielleadams pushed a commit that referenced this pull request Apr 11, 2023
Amend Tier 3 category definition to remove a
tentative clause and introduce a more concrete one.
Refs: #42802
PR-URL: #42805
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-failedAn error occurred while landing this pull request using GitHub Actions.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@gireeshpunathil@nodejs-github-bot@mcollina@cjihrig@mhdawson@RafaelGSS@RaisinTen