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: add optional RULE_DOCS_EXTENSION export for extended rule rationale #1229

Merged
merged 1 commit into from
Nov 24, 2022

Conversation

JamesHenry
Copy link
Member

@JamesHenry JamesHenry commented Nov 24, 2022

It's a real strength that we generate our rule documentation from both the rule implementations and unit tests. However, ESLint has a limited concept of rule documentation within the rule implementation itself - only a concise description is intended to be stored there.

Therefore, this PR introduces a simple RULE_DOCS_EXTENSION which a rule implementation file can optionally export and which will be used to generated a more extended rule rationale

Closes #1226

@codecov
Copy link

codecov bot commented Nov 24, 2022

Codecov Report

Merging #1229 (ada6cd4) into main (7b7bd76) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1229   +/-   ##
=======================================
  Coverage   88.07%   88.08%           
=======================================
  Files         162      162           
  Lines        3145     3146    +1     
  Branches      508      508           
=======================================
+ Hits         2770     2771    +1     
  Misses        261      261           
  Partials      114      114           
Flag Coverage Δ
unittest 88.08% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...lint-plugin-template/src/rules/no-negated-async.ts 100.00% <ø> (ø)

@JamesHenry JamesHenry merged commit 299a0c2 into main Nov 24, 2022
@JamesHenry JamesHenry deleted the rule-docs-extension branch November 24, 2022 09:52
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.

[no-negated-async] Add explanation.
1 participant