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

feat: add template-no-any rule #755

Merged
merged 3 commits into from
Feb 13, 2019
Merged

feat: add template-no-any rule #755

merged 3 commits into from
Feb 13, 2019

Conversation

rafaelss95
Copy link
Collaborator

@rafaelss95 rafaelss95 commented Feb 11, 2019

Please, check if the description, rationale and failure are clear enough :)

Issue: #633

@@ -29,6 +29,7 @@ export { Rule as PreferOutputReadonlyRule } from './preferOutputReadonlyRule';
export { Rule as TemplateConditionalComplexityRule } from './templateConditionalComplexityRule';
export { Rule as TemplateCyclomaticComplexityRule } from './templateCyclomaticComplexityRule';
export { Rule as TemplateAccessibilityTabindexNoPositiveRule } from './templateAccessibilityTabindexNoPositiveRule';
export { Rule as TemplateNoAnyRule } from './templateNoAnyRule';
Copy link
Owner

Choose a reason for hiding this comment

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

Would you also add the rule to the readme?

Copy link
Owner

@mgechev mgechev left a comment

Choose a reason for hiding this comment

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

Excellent! Would you only add this rule to the readme?

@rafaelss95
Copy link
Collaborator Author

rafaelss95 commented Feb 11, 2019

Excellent! Would you only add this rule to the readme?

Isn't it better to add it to README only when we publish a major version?

@wKoza
Copy link
Collaborator

wKoza commented Feb 12, 2019

@rafaelss95 , maybe, can you change the Readme of this branch ?

@mgechev mgechev merged commit 77a5e32 into mgechev:master Feb 13, 2019
@rafaelss95 rafaelss95 deleted the feat/template-no-any branch February 13, 2019 23: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.

3 participants