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(tslint): fix tslint config & errors #5747

Merged
merged 6 commits into from Apr 26, 2018
Merged

Conversation

manoldonev
Copy link
Contributor

Fix tslint.json syntax for some rules and the tslint errors uncovered by that fix.

@manoldonev manoldonev added this to the 4.1 milestone Apr 26, 2018
@manoldonev manoldonev self-assigned this Apr 26, 2018
@ns-bot
Copy link

ns-bot commented Apr 26, 2018

Can one of the admins verify this patch?

@ghost ghost added the in progress label Apr 26, 2018
"check-open-brace",
"check-catch",
"check-else",
"check-whitespace"
],
"quotemark": [false, "double"],
"quotemark": [
false,
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be true if we want to force double quotes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sis0k0 I did not disable this rule originally but it is now enabled and the errors fixed :).

@manoldonev
Copy link
Contributor Author

@sis0k0 enabled additionally arrow-return-shorthand and whitespace rules.

@manoldonev manoldonev merged commit 03cfc0c into master Apr 26, 2018
@ghost ghost removed the in progress label Apr 26, 2018
@manoldonev manoldonev deleted the mdonev/tslint-errors-fix branch April 26, 2018 15:36
@lock
Copy link

lock bot commented Aug 26, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Aug 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants