Skip to content

Commit

Permalink
doc: update code language flag for internal doc
Browse files Browse the repository at this point in the history
Use `bash` instead of `shell` for code language flag in
doc/guides/maintaining-ngtcp2-nghttp3.md to conform with our other docs
and upcoming lint requirements.

PR-URL: #33852
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Daijiro Wachi <[email protected]>
  • Loading branch information
Trott authored and codebytere committed Jul 12, 2020
1 parent 6651bde commit fbd6fe5
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions tools/icu/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ internationalization functionality.
* Configure Node.js with the specific [ICU version](http://icu-project.org/download)
you want to upgrade to, for example:

```shell
```bash
./configure \
--with-intl=small-icu \
--with-icu-source=http://download.icu-project.org/files/icu4c/58.1/icu4c-58_1-src.tgz
Expand All @@ -42,7 +42,7 @@ make

* Verify the Node.js build works:

```shell
```bash
make test-ci
```

Expand All @@ -58,13 +58,13 @@ new Intl.DateTimeFormat('es', {month: 'long'}).format(new Date(9E8));

* Now, copy `deps/icu` over to `deps/icu-small`

```shell
```bash
python tools/icu/shrink-icu-src.py
```

* Now, do a clean rebuild of Node.js to test:

```shell
```bash
make -k distclean
./configure
make
Expand All @@ -86,7 +86,7 @@ so make this a separate commit from the smaller changes.

* Now, rebuild the Node.js license.

```shell
```bash
# clean up - remove deps/icu
make clean
tools/license-builder.sh
Expand All @@ -96,7 +96,7 @@ tools/license-builder.sh
It should match the ICU URL used in the first step. When this is done, the
following should build with full ICU.

```shell
```bash
# clean up
rm -rf out deps/icu deps/icu4c*
./configure --with-intl=full-icu --download=all
Expand Down

0 comments on commit fbd6fe5

Please sign in to comment.