You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The new-rule tool (npm run new) should take a second (required) command line parameter asking for the author name/username and fill that into the template automatically. Currently, many new contributors ignore that field and it needs to be raised in the review.
The Requirements, Helpers and Rule Definitioncomment sections are sometimes hard to follow and their benefit is negligible. Also, most contributors ignore them completely, so it needs to be mentioned manually in the review. I suggest to drop them in the new-rule tool and in every existing rule.
The text was updated successfully, but these errors were encountered:
new-rule
tool (npm run new
) should take a second (required) command line parameter asking for the author name/username and fill that into the template automatically. Currently, many new contributors ignore that field and it needs to be raised in the review.Theactually seems to be already implemented.update-docs-rules-index
tool (included innpm run update
) should also automatically update the list of extension rules in https://eslint.vuejs.org/rules/#extension-rules. Noticed in #1996 (comment).Requirements
,Helpers
andRule Definition
comment sections are sometimes hard to follow and their benefit is negligible. Also, most contributors ignore them completely, so it needs to be mentioned manually in the review. I suggest to drop them in thenew-rule
tool and in every existing rule.The text was updated successfully, but these errors were encountered: