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

linter: implement from_configuration for jsx-a11y anchor-is-valid #2425

Merged
merged 3 commits into from
Feb 20, 2024

Conversation

yossydev
Copy link

@yossydev yossydev commented Feb 18, 2024

Created the from_configuration function to recognize links written during testing as valid. This function takes a JSON configuration and generates an AnchorIsValidConfig object, which determines whether specific hrefs are valid. This change allows us to easily specify valid hrefs in our tests, ensuring that our link validation logic works as expected.

@github-actions github-actions bot added the A-linter Area - Linter label Feb 18, 2024
Copy link

codspeed-hq bot commented Feb 18, 2024

CodSpeed Performance Report

Merging #2425 will not alter performance

Comparing yossydev:fix/jsx-a11y-anchor-is-valid (2ec5e67) with main (a2c173d)

Summary

✅ 27 untouched benchmarks

@yossydev yossydev force-pushed the fix/jsx-a11y-anchor-is-valid branch 3 times, most recently from b267c5c to e85e0ce Compare February 18, 2024 07:15
@yossydev yossydev marked this pull request as ready for review February 18, 2024 07:28
@Boshen
Copy link
Member

Boshen commented Feb 20, 2024

Let's merge first.

@Boshen Boshen merged commit e7ce633 into oxc-project:main Feb 20, 2024
20 checks passed
IWANABETHATGUY pushed a commit to IWANABETHATGUY/oxc that referenced this pull request May 29, 2024
…c-project#2425)

- fix: oxc-project#2361 

Created the `from_configuration` function to recognize links written
during testing as valid. This function takes a JSON configuration and
generates an `AnchorIsValidConfig` object, which determines whether
specific hrefs are valid. This change allows us to easily specify valid
hrefs in our tests, ensuring that our link validation logic works as
expected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linter Area - Linter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

linter: Implement from_configuration() for jsx_a11y/anchor-is-valid
3 participants