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

src: fix #endif description in crypto_keygen.h #40639

Conversation

tniessen
Copy link
Member

This should be the same as the condition at the beginning of the file.

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. labels Oct 27, 2021
@tniessen tniessen added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 27, 2021
@github-actions
Copy link
Contributor

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

@VoltrexKeyva VoltrexKeyva removed the needs-ci PRs that need a full CI run. label Oct 28, 2021
@jasnell jasnell added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 28, 2021
@github-actions github-actions 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 28, 2021
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/40639
✔  Done loading data for nodejs/node/pull/40639
----------------------------------- PR info ------------------------------------
Title      src: fix #endif description in crypto_keygen.h (#40639)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     tniessen:src-fix-endif-description-crypto-keygen -> nodejs:master
Labels     crypto, c++, fast-track
Commits    1
 - src: fix #endif description in crypto_keygen.h
Committers 1
 - Tobias Nießen 
PR-URL: https://github.com/nodejs/node/pull/40639
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Voltrex 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40639
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Voltrex 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Wed, 27 Oct 2021 22:35:52 GMT
   ✔  Approvals: 4
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/40639#pullrequestreview-791257072
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/40639#pullrequestreview-791286828
   ✔  - Voltrex (@VoltrexMaster): https://github.com/nodejs/node/pull/40639#pullrequestreview-791314930
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/40639#pullrequestreview-791939002
   ℹ  This PR is being fast-tracked
   ✔  Last GitHub Actions successful
   ✖  No Jenkins CI runs detected
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1395031492

VoltrexKeyva pushed a commit that referenced this pull request Oct 28, 2021
PR-URL: #40639
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@VoltrexKeyva
Copy link
Member

Landed in f3125cf

targos pushed a commit that referenced this pull request Nov 6, 2021
PR-URL: #40639
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos targos mentioned this pull request Nov 8, 2021
BethGriggs pushed a commit that referenced this pull request Nov 25, 2021
PR-URL: #40639
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@BethGriggs BethGriggs mentioned this pull request Nov 26, 2021
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Issues and PRs that require attention from people who are familiar with C++. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. crypto Issues and PRs related to the crypto subsystem. 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.

None yet

7 participants