Skip to content

Commit

Permalink
Rename build scripts to prepare (#70)
Browse files Browse the repository at this point in the history
  • Loading branch information
RomainMuller authored and Elad Ben-Israel committed Jul 11, 2018
1 parent b8b5086 commit a5a9270
Show file tree
Hide file tree
Showing 29 changed files with 4,929 additions and 4,811 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ using [lerna](https://github.com/lerna/lerna).

All modules within this repository have the following scripts:

* `prepare` - builds the module (usually runs the TypeScript compiler).
* `build` - builds the module (usually runs the TypeScript compiler).
* `watch` - runs `tsc -w` which picks up changes and builds them progressively.
* `test` - uses `nodeunit test/test.*.js` to run all unit tests.

Expand Down
4 changes: 2 additions & 2 deletions build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ rm -rf $BUILD_INDICATOR
export PATH=node_modules/.bin:$PATH

echo "============================================================================================="
echo "boostrapping..."
lerna bootstrap --reject-cycles
echo "building..."
lerna exec --stream npm run build

echo "============================================================================================="
echo "testing..."
Expand Down
4 changes: 4 additions & 0 deletions install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,7 @@ set -euo pipefail
echo "============================================================================================="
echo "installing repo-global dependencies..."
npm i --no-package-lock --global-style

echo "============================================================================================="
echo "boostrapping..."
lerna bootstrap --reject-cycles
Loading

1 comment on commit a5a9270

@RomainMuller
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The title of this commit has the script names mixed up -- it renamed prepare to build and not the other way around.

Please sign in to comment.