Skip to content

Add CI workflow to run Jest tests on PRs#395

Open
Copilot wants to merge 8 commits intomainfrom
copilot/analyze-test-coverage
Open

Add CI workflow to run Jest tests on PRs#395
Copilot wants to merge 8 commits intomainfrom
copilot/analyze-test-coverage

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 3, 2026

  • Resolve merge conflicts in package.json (combined main's new scripts/deps with our "test": "jest" + jest devDependencies)
  • Regenerate package-lock.json via npm install to include jest packages
  • Finalize merge commit

@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 3, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedjest-environment-jsdom@​29.7.01001006694100
Addedjest@​29.7.01001006894100

View full report

@jbampton jbampton marked this pull request as ready for review April 3, 2026 06:49
@jbampton jbampton self-requested a review as a code owner April 3, 2026 06:49
@jbampton jbampton requested a review from BaseMax April 3, 2026 06:51
@jbampton
Copy link
Copy Markdown
Member

jbampton commented Apr 3, 2026

/gemini review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a Jest testing suite, including configuration, package scripts, and unit tests for build data, Eleventy configuration, and utility functions. A review comment suggests adding the --coverage flag to the test script to ensure coverage metrics are collected during execution.

Co-authored-by: jbampton <418747+jbampton@users.noreply.github.com>
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