Skip to content

doc: mention v18.x openssl maintaining guide #45070

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged

Conversation

RafaelGSS
Copy link
Member

Nothing really changed. However, it's important to ensure main now refers to the 'Current' release, v19.x.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 19, 2022
Copy link
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

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

LGTM ✨

@lpinca
Copy link
Member

lpinca commented Oct 20, 2022

Is there a typo in commit message? "migration" -> "maintaining".

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@RafaelGSS RafaelGSS changed the title doc: mention v18.x openssl migration guide doc: mention v18.x openssl maintaining guide Oct 21, 2022
@RafaelGSS RafaelGSS force-pushed the doc/maintaining-openssl-v18 branch from 12c1c01 to e5b0dcd Compare October 21, 2022 12:57
@jasnell jasnell added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 21, 2022
@nodejs-github-bot nodejs-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 Oct 21, 2022
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/45070
✔  Done loading data for nodejs/node/pull/45070
----------------------------------- PR info ------------------------------------
Title      doc: mention v18.x openssl maintaining guide (#45070)
Author     Rafael Gonzaga  (@RafaelGSS)
Branch     RafaelGSS:doc/maintaining-openssl-v18 -> nodejs:main
Labels     doc
Commits    1
 - doc: mention v18.x openssl maintaining guide
Committers 1
 - RafaelGSS 
PR-URL: https://github.com/nodejs/node/pull/45070
Reviewed-By: Antoine du Hamel 
Reviewed-By: Richard Lau 
Reviewed-By: Beth Griggs 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Michael Dawson 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/45070
Reviewed-By: Antoine du Hamel 
Reviewed-By: Richard Lau 
Reviewed-By: Beth Griggs 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Michael Dawson 
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last review:
   ⚠  - doc: mention v18.x openssl maintaining guide
   ℹ  This PR was created on Wed, 19 Oct 2022 12:26:25 GMT
   ✔  Approvals: 6
   ✔  - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/45070#pullrequestreview-1147464514
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/45070#pullrequestreview-1147480751
   ✔  - Beth Griggs (@BethGriggs) (TSC): https://github.com/nodejs/node/pull/45070#pullrequestreview-1147605979
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/45070#pullrequestreview-1148304898
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/45070#pullrequestreview-1149876771
   ✔  - Michael Dawson (@mhdawson) (TSC): https://github.com/nodejs/node/pull/45070#pullrequestreview-1150058050
   ✔  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/3298570699

@richardlau richardlau 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. labels Oct 21, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 21, 2022
@nodejs-github-bot nodejs-github-bot merged commit e43d191 into nodejs:main Oct 21, 2022
@nodejs-github-bot
Copy link
Collaborator

Landed in e43d191

RafaelGSS added a commit that referenced this pull request Nov 1, 2022
PR-URL: #45070
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
@RafaelGSS RafaelGSS mentioned this pull request Nov 1, 2022
RafaelGSS added a commit that referenced this pull request Nov 10, 2022
PR-URL: #45070
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
danielleadams pushed a commit that referenced this pull request Dec 30, 2022
PR-URL: #45070
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
danielleadams pushed a commit that referenced this pull request Dec 30, 2022
PR-URL: #45070
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
danielleadams pushed a commit that referenced this pull request Jan 3, 2023
PR-URL: #45070
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants