FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

build: expand ASAN acronym in configure help by sam-github · Pull Request #32325 · nodejs/node · GitHub

/ node Public

build: expand ASAN acronym in configure help - #32325

Closed
sam-github wants to merge 1 commit into
nodejs:masterfrom
sam-github:asan-expand
Closed

build: expand ASAN acronym in configure help#32325
sam-github wants to merge 1 commit into
nodejs:masterfrom
sam-github:asan-expand

Conversation

sam-github commented Mar 17, 2020
edited
Loading

Copy link
Copy Markdown
Contributor

There is lots of space to expand this not-so-well-known acronym in the
option usage message.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

nodejs-github-bot added the build Issues and PRs related to build files or the CI. label Mar 17, 2020
Comment thread configure.py Outdated
There is lots of space to expand this not-so-well-known acronym in the
option usage message.

Copy link
Copy Markdown
Contributor Author

Landed in 131b50d

sam-github closed this Mar 20, 2020
sam-github deleted the asan-expand branch March 20, 2020 21:29
sam-github added a commit that referenced this pull request Mar 20, 2020
There is lots of space to expand this not-so-well-known acronym in the
option usage message.

PR-URL: #32325
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Matheus Marchini <mat@mmarchini.me>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Mar 24, 2020
There is lots of space to expand this not-so-well-known acronym in the
option usage message.

PR-URL: #32325
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Matheus Marchini <mat@mmarchini.me>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins mentioned this pull request Mar 24, 2020
targos pushed a commit that referenced this pull request Apr 22, 2020
There is lots of space to expand this not-so-well-known acronym in the
option usage message.

PR-URL: #32325
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Matheus Marchini <mat@mmarchini.me>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to build files or the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants


Back | FazBrowse Home | New Git URL