7fe8052941
After 233c5a8
landed, I noticed there are a few things to be fixed up:
* Run gometalinter in all the right places. We need to run both in
lint and lint_quiet targets. I've also cleaned up some of the logic
around what to suppress so there's less repetition.
* We currently @ meaningful commands, which is unfortunate, since it
makes debugging Makefiles tough (especially when looking at CI build
logs). Going forward, we should only use @ for meaningless commands,
like @echo.
* The AWS project wasn't actually running tslint, because it needs to
say `tslint './pack/**/*.ts' --exclude='./pack/node_modules/**'`.
The current script of `tslint lib/aws/pack/...` wasn't actually
running lint, hence we missed a lot of AWS lint issues.
* Fix up the issues that these fixes uncovered. Mostly err shadowing.
35 lines
575 B
Makefile
35 lines
575 B
Makefile
INSTALL=/usr/local/bin
|
|
|
|
.PHONY: default
|
|
default: banner build test
|
|
|
|
.PHONY: banner
|
|
banner:
|
|
@echo "\033[1;37m===============\033[0m"
|
|
@echo "\033[1;37mLumiJS Compiler\033[0m"
|
|
@echo "\033[1;37m===============\033[0m"
|
|
|
|
.PHONY: clean
|
|
clean:
|
|
rm -rf ./bin
|
|
rm -rf ${INSTALL}
|
|
|
|
.PHONY: build
|
|
build:
|
|
@echo "\033[0;32mBUILD:\033[0m"
|
|
yarn run build
|
|
|
|
.PHONY: install
|
|
install:
|
|
@echo "\033[0;32mINSTALL:\033[0m"
|
|
ln -sf ${shell pwd}/lumijs ${INSTALL}/lumijs
|
|
|
|
.PHONY: test
|
|
test:
|
|
@echo "\033[0;32mTEST:\033[0m"
|
|
yarn run test
|
|
|
|
.PHONY: verify
|
|
verify: gen
|
|
$(shell git diff --quiet .)
|
|
|