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] label-has-associated-control: allow labelComponents to contain globs #1025

Conversation

lb-
Copy link
Contributor

@lb- lb- commented Oct 23, 2024

Add ability for labelComponents within the label-has-associated-control to use the same glob checking mechanism as controlComponents.

  • Ensure existing tests pass and update unit tests for new behaviour
  • Add extra tests for documented ???Foo syntax for component glob matching
  • Update documentation to have appropriate examples for label/control glob usage
  • Closes #972

Builds on PR #1024

…ain globs

Add ability for `labelComponents` within the `label-has-associated-control` to use the same glob checking mechanism as `controlComponents`.

- Ensure existing tests pass and update unit tests for new behaviour
- Add extra tests for documented `???Foo` syntax for component glob matching
- Update documentation to have appropriate examples for label/control glob usage

Closes jsx-eslint#972
@lb-
Copy link
Contributor Author

lb- commented Oct 25, 2024

Note to self. Need to rebase to resolve conflicts.

@lb- lb- force-pushed the feature/972-label-has-associated-control-glob-label-component branch from 16e355b to d84726f Compare October 26, 2024 06:50
@ljharb ljharb force-pushed the feature/972-label-has-associated-control-glob-label-component branch from d84726f to 743168b Compare October 27, 2024 05:47
@ljharb ljharb changed the title [feature] Add labelComponents glob checks in label-has-associated-control [New] label-has-associated-control: allow labelComponents to contain globs Oct 27, 2024
@ljharb ljharb merged commit 743168b into jsx-eslint:main Oct 27, 2024
112 checks passed
@lb- lb- deleted the feature/972-label-has-associated-control-glob-label-component branch October 27, 2024 19:58
@lb-
Copy link
Contributor Author

lb- commented Oct 27, 2024

Thank you @ljharb

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

Successfully merging this pull request may close these issues.

2 participants