Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Run ts-lint as part of the CI #532

Closed
vicb opened this issue Dec 8, 2016 · 1 comment · Fixed by #565
Closed

Run ts-lint as part of the CI #532

vicb opened this issue Dec 8, 2016 · 1 comment · Fixed by #565

Comments

@vicb
Copy link
Contributor

vicb commented Dec 8, 2016

see #13316

  • catch syntax errors,
  • catch formatting errors (clang-format)
JiaLiPassion added a commit to JiaLiPassion/zone.js that referenced this issue Dec 27, 2016
@JiaLiPassion
fix angular#532, add tslint in ci, add tslint/karma/testnode/format i…
d28189f 
…n git commit hook
JiaLiPassion added a commit to JiaLiPassion/zone.js that referenced this issue Dec 27, 2016
@JiaLiPassion
fix angular#532, add tslint in ci, add tslint/karma/testnode/format i…
979595f 
…n git commit hook
@JiaLiPassion JiaLiPassion mentioned this issue Dec 27, 2016
Merged
@vicb
Copy link
Contributor Author

vicb commented Dec 28, 2016

closed by #550

@vicb vicb closed this as completed Dec 28, 2016
mhevery pushed a commit that referenced this issue Jan 12, 2017
@JiaLiPassion @mhevery
Fix #532, Fix #566, add tslint in ci, add tslint/format/test/karma in…
fb8d51c 
… precommit of git (#565)

* fix #532, add tslint in ci, add tslint/karma/testnode/format in git commit hook

* fix #532, add tslint in ci, add tslint/karma/testnode/format in git commit hook

* use format:enforce

* rename package.json script's name

* update developer.md to add precommit details

* change var to const

* add scripts explaination into DEVELOPER.md, use gulp lint

* remove pre-commit hooks

* update DEVELOPER.md to add process for before commit

* format DEVELOPMER.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@vicb