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

Adding support for a white list of elements that can be lower case. #188

Merged
merged 5 commits into from
Sep 3, 2018
Merged

Adding support for a white list of elements that can be lower case. #188

merged 5 commits into from
Sep 3, 2018

Conversation

aaronkahlhamer
Copy link
Contributor

When enabling the rule instead of using a boolean, use an array with the attributes (camelCased) that you want to ignore

{
  "tagname-lowercase": ["clipPath"]
}

When enabling the rule instead of using a boolean, use an array with the attributes (camelCased) that you want to ignore
```json
{
  "tagname-lowercase": ["clipPath"]
}
```
@coveralls
Copy link

coveralls commented Jan 17, 2017

Coverage Status

Coverage increased (+0.002%) to 98.736% when pulling ccfdae5 on aaronkahlhamer:patch-1 into 152a114 on yaniswang:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 98.736% when pulling ccfdae5 on aaronkahlhamer:patch-1 into 152a114 on yaniswang:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 98.736% when pulling ccfdae5 on aaronkahlhamer:patch-1 into 152a114 on yaniswang:master.

@hacknug
Copy link

hacknug commented Aug 22, 2018

@thedaviddias imho this branch should also be merged asap. Documentation should be added but we can create a new issue for that.

@thedaviddias
Copy link
Member

Ok @hacknug, I'm going to review and accept it. Thanks!

@thedaviddias thedaviddias self-requested a review August 22, 2018 14:57
@thedaviddias thedaviddias self-assigned this Aug 22, 2018
@thedaviddias thedaviddias added enhancement Functionality that enhances existing features #status: dev in progress labels Aug 22, 2018
@Shinigami92
Copy link
Contributor

This will solve #263 👍

But @aaronkahlhamer can you please add one or more tests to cover this case?

Thank you for your contribution

@thedaviddias thedaviddias changed the base branch from master to develop August 26, 2018 20:17
@codecov-io
Copy link

codecov-io commented Aug 26, 2018

Codecov Report

Merging #188 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           develop   #188   +/-   ##
======================================
  Coverage      100%   100%           
======================================
  Files            1      1           
  Lines            1      1           
======================================
  Hits             1      1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8e46d30...e2707e2. Read the comment docs.

@thedaviddias thedaviddias merged commit 5cf8348 into htmlhint:develop Sep 3, 2018
thedaviddias pushed a commit that referenced this pull request Sep 3, 2018
… be lower case. (#188)

When enabling the rule instead of using a boolean, use an array with the attributes (camelCased) that you want to ignore
```json
{
  "tagname-lowercase": ["clipPath"]
}
```
thedaviddias pushed a commit that referenced this pull request Sep 3, 2018
… be lower case. (#188)

When enabling the rule instead of using a boolean, use an array with the attributes (camelCased) that you want to ignore
```json
{
  "tagname-lowercase": ["clipPath"]
}
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Functionality that enhances existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants