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.

Sign up for GitHub

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

Jump to bottom

[New] add requireDataLowercase option to no-unknown-property rule #3645

Merged
merged 1 commit into from
Nov 9, 2023
Merged

[New] add requireDataLowercase option to no-unknown-property rule #3645

merged 1 commit into from
Nov 9, 2023

Conversation

HermanBilous
Copy link
Contributor

Fixes #3643

@codecov
Copy link

codecov bot commented Oct 20, 2023 •
edited
Loading

Codecov Report

Merging #3645 (eaf977b) into master (ca30f77) will not change coverage.
The diff coverage is 100.00%.

❗ Current head eaf977b differs from pull request most recent head 70e6019. Consider uploading reports for the commit 70e6019 to get more accurate results

@@           Coverage Diff           @@
##           master    #3645   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files         132      132           
  Lines        9369     9369           
  Branches     3429     3431    +2     
=======================================
  Hits         9149     9149           
  Misses        220      220
Files Coverage Δ
lib/rules/no-unknown-property.js 98.93% <100.00%> (+0.11%) ⬆️

... and 4 files with indirect coverage changes

ljharb
ljharb requested changes Oct 21, 2023
View reviewed changes
lib/rules/no-unknown-property.js Outdated Show resolved Hide resolved
tests/lib/rules/no-unknown-property.js Outdated Show resolved Hide resolved
@HermanBilous HermanBilous requested a review from ljharb October 23, 2023 13:54
@HermanBilous HermanBilous requested a review from ljharb October 24, 2023 09:55
@HermanBilous HermanBilous changed the title [New] add allowDataUpperCase option to no-unknown-property rule [New] add requireDataLowercase option to no-unknown-property rule Oct 24, 2023
@HermanBilous
Copy link
Contributor Author

Hey, @ljharb, could you take a look? I addressed all your comments.

ljharb
ljharb reviewed Oct 30, 2023
View reviewed changes
docs/rules/no-unknown-property.md Outdated Show resolved Hide resolved
lib/rules/no-unknown-property.js Outdated Show resolved Hide resolved
@HermanBilous HermanBilous requested a review from ljharb November 8, 2023 09:11
@ljharb ljharb merged commit 70e6019 into jsx-eslint:master Nov 9, 2023
311 checks passed
@BuloZB BuloZB mentioned this pull request May 19, 2024
Open
@Exkaleburx Exkaleburx mentioned this pull request May 24, 2024
Open
@Balantion2020 Balantion2020 mentioned this pull request May 24, 2024
Open
This was referenced May 24, 2024
Open
Open
Open
@mjperrins mjperrins mentioned this pull request May 30, 2024
Open
@richyhoopd richyhoopd mentioned this pull request Jun 3, 2024
Open
@Luis-Kristen Luis-Kristen mentioned this pull request Jun 3, 2024
Open
@Exkaleburx Exkaleburx mentioned this pull request Jun 3, 2024
Open
@Balantion2020 Balantion2020 mentioned this pull request Jun 3, 2024
Open
This was referenced Jun 3, 2024
Open
Open
@Tanver-Hasan Tanver-Hasan mentioned this pull request Jun 4, 2024
Open
@ManuelDevWeb ManuelDevWeb mentioned this pull request Jun 5, 2024
Open
@cezaraugusto cezaraugusto mentioned this pull request Jun 6, 2024
Merged
@paaschdigital paaschdigital mentioned this pull request Jun 7, 2024
Open
This was referenced Oct 3, 2024
Open
Open
@richyhoopd richyhoopd mentioned this pull request Oct 3, 2024
Open
This was referenced Oct 3, 2024
Open
Open
Open
@Exkaleburx Exkaleburx mentioned this pull request Oct 3, 2024
Open
This was referenced Oct 3, 2024
Open
Open
Open
@scatools-demo scatools-demo mentioned this pull request Oct 3, 2024
Open
@Luis-Kristen Luis-Kristen mentioned this pull request Oct 4, 2024
Open
@DadonStyle DadonStyle mentioned this pull request Oct 4, 2024
Open
@Tanver-Hasan Tanver-Hasan mentioned this pull request Oct 4, 2024
Open
@zdr4k zdr4k mentioned this pull request Oct 4, 2024
Open
@Balantion2020 Balantion2020 mentioned this pull request Oct 4, 2024
Open
This was referenced Oct 4, 2024
Open
Open
Open
@richyhoopd richyhoopd mentioned this pull request Oct 4, 2024
Open
This was referenced Oct 4, 2024
Open
Open
@scatools-demo scatools-demo mentioned this pull request Oct 4, 2024
Open
This was referenced Oct 4, 2024
Open
Open
@Exkaleburx Exkaleburx mentioned this pull request Oct 4, 2024
Open
@CASABECI CASABECI mentioned this pull request Oct 6, 2024
Open
@scatools-demo scatools-demo mentioned this pull request Oct 6, 2024
Open
@Sarrac3873 Sarrac3873 mentioned this pull request Oct 16, 2024
Open
@MANAALHARBI MANAALHARBI mentioned this pull request Dec 2, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ljharb ljharb ljharb approved these changes

Assignees
No one assigned
Labels
enhancement
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug]: no-unknown-property gives false-positives on data-* attributes, but React gives warnings on them
2 participants
@HermanBilous @ljharb