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

[New] jsx-pascal-case: support namespace option #2917

Merged
merged 1 commit into from Feb 23, 2021
Merged

[New] jsx-pascal-case: support namespace option #2917

merged 1 commit into from Feb 23, 2021

Conversation

Kev-Y-Huang
Copy link
Contributor

Fixes #2914.

New feature allows users to ignore text after the dot for namespaced components such as Theme-UI's Styled component <Styled.h1>.

As part of the changes, also addressed some issues that I identified in the rule. Previously if a component was namespaced the text before the dot could be not camelcase and could even be non-alphanumeric.

@ehowey
Copy link

ehowey commented Feb 5, 2021

Thanks for your work on this Ken! I want to be rid of these errors as quickly as possible!

lib/rules/jsx-pascal-case.js Outdated Show resolved Hide resolved
@Kev-Y-Huang
Copy link
Contributor Author

Hopefully my news changes are a bit more elegant than using error handling to implement the rule @ljharb

@LekoArts
Copy link

LekoArts commented Mar 5, 2021

Thanks for the PR @Kev-Y-Huang 👍

Hi @ljharb 👋 I sadly couldn't find any information on when you're releasing stuff. When can we expect this PR to be released / alternatively is there a nightly release?

With the documentation merged I assumed this is already published. Thanks!

@ljharb
Copy link
Member

ljharb commented Mar 5, 2021

There is certainly no nightly release, and no schedule. It will be in the next release.

Merged never automatically implies released, in any project on GitHub. You should be checking the documentation on the latest tag for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

react/jsx-pascal-case flags Styled.h1 as not being in pascal case
4 participants