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.

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

chore: updated tslint #1021

Merged
merged 1 commit into from
Sep 10, 2018
Merged

chore: updated tslint #1021

merged 1 commit into from
Sep 10, 2018

Conversation

jdx
Copy link
Contributor

@jdx jdx commented Sep 7, 2018

cc @elbandito

this just updates the linter rules to be a bit stricter

@jdx jdx requested review from a team and elbandito September 7, 2018 22:19
Copy link
Contributor

@amerine amerine left a comment

Choose a reason for hiding this comment

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

space changes LGTM

@msakrejda
Copy link
Contributor

Ignoring whitespace, this is a 10-line diff so +1 =D

@RasPhilCo
Copy link
Contributor

RasPhilCo commented Sep 7, 2018

LGTM!

Copy link
Contributor

@RasPhilCo RasPhilCo left a comment

Choose a reason for hiding this comment

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

🏄

@jdx jdx force-pushed the tslint branch 2 times, most recently from 1ff4524 to f52268b Compare September 7, 2018 23:20
@jdx jdx merged commit 62ea117 into master Sep 10, 2018
@jdx jdx deleted the tslint branch September 10, 2018 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants