Skip to content

Conversation

JounQin
Copy link
Member

@JounQin JounQin commented May 21, 2025

related #7

@JounQin JounQin requested a review from Copilot May 21, 2025 03:34
@JounQin JounQin self-assigned this May 21, 2025
Copy link

changeset-bot bot commented May 21, 2025

⚠️ No Changeset found

Latest commit: 03137f8

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates tests from Tape to Jest and updates associated configurations and documentation. Key changes include:

  • Replacing Tape assertions with Jest expectations in test files.
  • Updating documentation badges and workflow configurations.
  • Removing an obsolete code coverage configuration file.

Reviewed Changes

Copilot reviewed 85 out of 85 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/src/rules/aria-proptypes.spec.js Converted a Tape test to use Jest’s expect API for validityCheck.
tests/index.spec.js Updated tests to replace Tape assertions with Jest syntax.
README.md Revised badges and header formatting to improve documentation clarity.
.github/workflows/ci.yml Adjusted Node version configuration from fixed version to LTS markers.
.c8rc.json Removed the code coverage configuration file.
Comments suppressed due to low confidence (1)

.c8rc.json:1

  • The removal of the .c8rc.json file may affect code coverage reporting. Ensure that an alternative configuration is in place or update the documentation accordingly.
{

Copy link

codesandbox-ci bot commented May 21, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link

pkg-pr-new bot commented May 21, 2025

Open in StackBlitz

npm i https://pkg.pr.new/eslint-plugin-jsx-a11y-x@24

commit: 03137f8

Copy link

codecov bot commented May 21, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@JounQin JounQin force-pushed the test/jest branch 3 times, most recently from 22a5a22 to 7507105 Compare May 21, 2025 03:48
Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 1929c2e1
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1929c2e) Report Missing Report Missing Report Missing
Head commit (7507105) 1345 1333 99.11%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#24) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

Copy link

codacy-production bot commented May 21, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 1929c2e1
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1929c2e) Report Missing Report Missing Report Missing
Head commit (03137f8) 1345 1333 99.11%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#24) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

@JounQin JounQin merged commit d26b1e7 into main May 21, 2025
85 checks passed
@JounQin JounQin deleted the test/jest branch May 21, 2025 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant