Skip to content

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

Closed
tniessen wants to merge 1 commit into
nodejs:masterfrom
tniessen:src-fix-endif-description-crypto-keygen
Closed

src: fix #endif description in crypto_keygen.h#40639
tniessen wants to merge 1 commit into
nodejs:masterfrom
tniessen:src-fix-endif-description-crypto-keygen

Conversation

@tniessen

Copy link
Copy Markdown
Member

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

@nodejs-github-botnodejs-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
@tniessentniessen added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 27, 2021
@github-actions

Copy link
Copy Markdown
Contributor

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

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

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>
@targostargos 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>
@BethGriggsBethGriggs mentioned this pull request Nov 26, 2021
1 task
Sign up for freeto 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-failedAn error occurred while landing this pull request using GitHub Actions.cryptoIssues and PRs related to the crypto subsystem.fast-trackPRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@tniessen@VoltrexKeyva@jasnell@cjihrig@richardlau@bury9555@nodejs-github-bot