Skip to content

Apply tslint to services.ts file#6141

Merged
yuit merged 2 commits into
masterfrom
linting_services
Dec 18, 2015
Merged

Apply tslint to services.ts file#6141
yuit merged 2 commits into
masterfrom
linting_services

Conversation

@yuit
Copy link
Copy Markdown
Contributor

@yuit yuit commented Dec 18, 2015

Slowly applying lint rule to services folder as doing all at once will be quite insane 😅

Comment thread Jakefile.js Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Space after ","

@DanielRosenwasser
Copy link
Copy Markdown
Member

GitHub really sucks here.

@DanielRosenwasser
Copy link
Copy Markdown
Member

@DanielRosenwasser
Copy link
Copy Markdown
Member

There are a few places where you've touched the comments and switched let to const. That wasn't really necessary, and in the future, we should try to avoid that since it changes the last person associated with that line.

Anyhow, that's not all that big of a deal, so 👍

@yuit
Copy link
Copy Markdown
Contributor Author

yuit commented Dec 18, 2015

@DanielRosenwasser yeah. the list of lint errors are too big so I end up find-replace for let/const conversion 😅

yuit added a commit that referenced this pull request Dec 18, 2015
@yuit yuit merged commit 710a5be into master Dec 18, 2015
@yuit yuit deleted the linting_services branch December 18, 2015 02:26
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants