From 280ef172b87fde51233280b87ab238bab0c838bb Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Tue, 17 Sep 2019 13:59:32 -0700 Subject: [PATCH] doc: update list style in misc README docs To confirm with upcoming markdown lint rule, use `*` for unordered lists. PR-URL: https://github.com/nodejs/node/pull/29594 Reviewed-By: Richard Lau Reviewed-By: Colin Ihrig --- test/wpt/README.md | 6 +++--- tools/icu/README.md | 40 ++++++++++++++++++++-------------------- 2 files changed, 23 insertions(+), 23 deletions(-) diff --git a/test/wpt/README.md b/test/wpt/README.md index 15318b2aefe5d7..d15a56745be090 100644 --- a/test/wpt/README.md +++ b/test/wpt/README.md @@ -134,9 +134,9 @@ It also maintains the LICENSE file in `test/fixtures/wpt`. Given a module, the `WPTRunner` class in [`test/common/wpt`](../common/wpt.js) loads: -- `.js` test files (for example, `test/common/wpt/url/*.js` for `url`) -- Status file (for example, `test/wpt/status/url.json` for `url`) -- The WPT harness +* `.js` test files (for example, `test/common/wpt/url/*.js` for `url`) +* Status file (for example, `test/wpt/status/url.json` for `url`) +* The WPT harness Then, for each test, it creates a vm with the globals and mocks, sets up the harness result hooks, loads the metadata in the test (including diff --git a/tools/icu/README.md b/tools/icu/README.md index a2613544241b8e..94eaf50a55c82a 100644 --- a/tools/icu/README.md +++ b/tools/icu/README.md @@ -4,25 +4,25 @@ This directory contains tools, data, and information about the [ICU](http://icu- (International Components for Unicode) integration. ICU is used to provide internationalization functionality. -- `patches/` are one-off patches, actually entire source file replacements, +* `patches/` are one-off patches, actually entire source file replacements, organized by ICU version number. -- `icu_small.json` controls the "small" (English only) ICU. It is input to +* `icu_small.json` controls the "small" (English only) ICU. It is input to `icutrim.py` -- `icu-generic.gyp` is the build file used for most ICU builds within ICU. +* `icu-generic.gyp` is the build file used for most ICU builds within ICU. -- `icu-system.gyp` is an alternate build file used when `--with-intl=system-icu` +* `icu-system.gyp` is an alternate build file used when `--with-intl=system-icu` is invoked. It builds against the `pkg-config` located ICU. -- `iculslocs.cc` is source for the `iculslocs` utility, invoked by `icutrim.py` +* `iculslocs.cc` is source for the `iculslocs` utility, invoked by `icutrim.py` as part of repackaging. Not used separately. See source for more details. -- `no-op.cc` — empty function to convince gyp to use a C++ compiler. -- `README.md` — you are here -- `shrink-icu-src.py` — this is used during upgrade (see guide below) +* `no-op.cc` — empty function to convince gyp to use a C++ compiler. +* `README.md` — you are here +* `shrink-icu-src.py` — this is used during upgrade (see guide below) ## How to upgrade ICU -- Make sure your Node.js workspace is clean (clean `git status`) should be +* Make sure your Node.js workspace is clean (clean `git status`) should be sufficient. -- Configure Node.js with the specific [ICU version](http://icu-project.org/download) +* Configure Node.js with the specific [ICU version](http://icu-project.org/download) you want to upgrade to, for example: ```shell @@ -35,12 +35,12 @@ make > _Note_ in theory, the equivalent `vcbuild.bat` commands should work also, > but the commands below are makefile-centric. -- If there are ICU version-specific changes needed, you may need to make changes +* If there are ICU version-specific changes needed, you may need to make changes in `icu-generic.gyp` or add patch files to `tools/icu/patches`. - - Specifically, look for the lists in `sources!` in the `icu-generic.gyp` for + * Specifically, look for the lists in `sources!` in the `icu-generic.gyp` for files to exclude. -- Verify the Node.js build works: +* Verify the Node.js build works: ```shell make test-ci @@ -56,13 +56,13 @@ new Intl.DateTimeFormat('es', {month: 'long'}).format(new Date(9E8)); …Should return `January` not `enero`. -- Now, copy `deps/icu` over to `deps/icu-small` +* Now, copy `deps/icu` over to `deps/icu-small` ```shell python tools/icu/shrink-icu-src.py ``` -- Now, do a clean rebuild of Node.js to test: +* Now, do a clean rebuild of Node.js to test: ```shell make -k distclean @@ -70,7 +70,7 @@ make -k distclean make ``` -- Test this newly default-generated Node.js +* Test this newly default-generated Node.js @@ -84,7 +84,7 @@ new Intl.DateTimeFormat('es', {month: 'long'}).format(new Date(9E8)); You are ready to check in the updated `deps/small-icu`. This is a big commit, so make this a separate commit from the smaller changes. -- Now, rebuild the Node.js license. +* Now, rebuild the Node.js license. ```shell # clean up - remove deps/icu @@ -92,7 +92,7 @@ make clean tools/license-builder.sh ``` -- Update the URL and hash for the full ICU file in `tools/icu/current_ver.dep`. +* Update the URL and hash for the full ICU file in `tools/icu/current_ver.dep`. It should match the ICU URL used in the first step. When this is done, the following should build with full ICU. @@ -104,9 +104,9 @@ make make test-ci ``` -- commit the change to `tools/icu/current_ver.dep` and `LICENSE` files. +* commit the change to `tools/icu/current_ver.dep` and `LICENSE` files. - - Note: To simplify review, I often will “pre-land” this patch, meaning that + * Note: To simplify review, I often will “pre-land” this patch, meaning that I run the patch through `curl -L https://github.com/nodejs/node/pull/xxx.patch | git am -3 --whitespace=fix` per the collaborator’s guide… and then push that patched branch into my PR's branch. This reduces the whitespace changes that