-
-
Notifications
You must be signed in to change notification settings - Fork 385
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(rules): add html-lang-require rule #632
feat(rules): add html-lang-require rule #632
Conversation
test/rules/html-lang-require.spec.js
Outdated
ruleOptions[ruldId] = true | ||
|
||
describe(`Rules: ${ruldId}`, () => { | ||
it('Html tag have no a lang attribute should result in an error', () => { |
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.
Html
here should be HTML
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.
fixed
* added html-lang-require rule * add tests for html-lang-require rule
68b7764
to
1d946ff
Compare
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.
Thanks @baleyko for this, ideally the doc should also be updated but it can be done in another PR. Would you mind create another ticket to update the doc accordingly to this new rule? I let you squash and merge this ticket if you want.
# [0.15.0](v0.14.2...v0.15.0) (2021-06-10) ### Features * **rules:** add html-lang-require rule ([#632](#632)) ([51471a9](51471a9))
🎉 This PR is included in version 0.15.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This rule helps make HTML documents to correspond to the W3C requirements.
If the rule is enabled, the next example should be valid:
And the next one should not be valid: