-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Use link to tagged version for rule docs #1005
Conversation
src/docsUrl.js
Outdated
@@ -1,5 +1,7 @@ | |||
import pkg from '../package' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it'd be nice to use .json
here (unless the linter objects)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
b2a5024
to
8d203d8
Compare
tests/src/core/docsUrl.js
Outdated
@@ -1,13 +1,14 @@ | |||
import { expect } from 'chai' | |||
|
|||
import pkg from '../../../package' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also here :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea.. realised it after the first push 😅. Done.
8d203d8
to
fa24082
Compare
This uses the version present in package.json for generating the URL to rule documentation if no commit-ish value is passed. I think this is good because the documentation will match the version the user has installed. This way, even if a rule is changed or removed in the future, the user can find the documentation with ease.