Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

TSLint rules and fixes #3642

Closed
wants to merge 10 commits into from
Closed

TSLint rules and fixes #3642

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
82ecd5d
Basic tslint rules, non-final list
danquirk Jun 25, 2015
ed1ff3d
Fixing up whitespace and semicolons
danquirk Jun 25, 2015
d4403df
var -> let
danquirk Jun 26, 2015
cd48d81
Fix build errors due to naive var->let replacement
danquirk Jun 26, 2015
5fc3099
Rule changes
danquirk Jun 30, 2015
be1371d
Merge master
danquirk Jul 8, 2015
5dfa610
Turn off a few rules and more cleanup post merge
danquirk Jul 9, 2015
27f5bf1
CR feedback
danquirk Jul 9, 2015
a80a4ba
Add jake task for linting the compiler sources
danquirk Jul 9, 2015
c363dec
for...in not for...of
danquirk Jul 9, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
for...in not for...of
  • Loading branch information
@danquirk
danquirk committed Jul 9, 2015
commit c363dec9af72fe8397105e9704052b121e87795c
3 changes: 2 additions & 1 deletion Jakefile.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -730,7 +730,8 @@ task("update-sublime", [serverFile], function() {
// run this task automatically
desc("Runs tslint on the compiler sources");
task("lint", [], function() {
for(var f of compilerSources) {
for(var i in compilerSources) {
var f = compilerSources[i];
var cmd = 'tslint -f ' + f;
exec(cmd,
function() { console.log('SUCCESS: No linter errors'); },
Expand Down