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

[tslint] enable no-default-export rule #20952

Merged
merged 1 commit into from
Jul 19, 2018

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Jul 18, 2018

Just like our ESLint rules, I think we should enforce named exports over default exports via TSLint. No changes to the code were necessary because of this linting rule because everyone is already following the pattern we established with ESLint, but TSLint wasn't actually verifying the pattern.

@spalger spalger added review Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v7.0.0 v6.4.0 labels Jul 18, 2018
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit 29d8ec4 into elastic:master Jul 19, 2018
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 19, 2018
Just like our ESLint rules, I think we should enforce named exports over default exports via TSLint. No changes to the code were necessary because of this linting rule because everyone is already following the pattern we established with ESLint, but TSLint wasn't actually verifying the pattern.
spalger pushed a commit that referenced this pull request Jul 19, 2018
Backports the following commits to 6.x:
 - [tslint] enable no-default-export rule  (#20952)
@spalger
Copy link
Contributor Author

spalger commented Jul 19, 2018

6.x/6.4: 17694da

@spalger spalger deleted the tslint/no-default-export branch July 19, 2018 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v6.4.0 v7.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants