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

Remove dequal as a dep from the project. #357

Merged
merged 1 commit into from
Jul 1, 2024
Merged

Conversation

samccone
Copy link
Contributor

@samccone samccone commented Jul 1, 2024

Inspection of the code and types showed that the actual check does not need to handle all cases, but rather there is a small handful of attributes and cases to check. As such do it in the project directly and remove the entire need for a dep.

Ref: #354

Inspection of the code and types showed that the actual check does not need to handle all cases, but rather there is a small handful of attributes and cases to check. As such do it in the project and remove the entire need for a dep.

Ref: A11yance#354
ljharb
ljharb previously approved these changes Jul 1, 2024
Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, as discussed, this seems like a much simpler approach, and simultaneously resolves both engines concerns as well as dep count/bundle size concerns.

I'll backport this to v3, as well.

@ljharb ljharb merged commit 39030e7 into A11yance:main Jul 1, 2024
6 checks passed
ljharb pushed a commit that referenced this pull request Jul 1, 2024
Inspection of the code and types showed that the actual check does not need to handle all cases, but rather there is a small handful of attributes and cases to check. As such do it in the project and remove the entire need for a dep.

Ref: #354
@coveralls
Copy link

coveralls commented Jul 1, 2024

Pull Request Test Coverage Report for Build 9750552373

Details

  • 11 of 12 (91.67%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.4%) to 96.764%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/elementAXObjectMap.js 11 12 91.67%
Totals Coverage Status
Change from base Build 9707852721: -0.4%
Covered Lines: 233
Relevant Lines: 235

💛 - Coveralls

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.

3 participants