Skip to content

ci: add jest workflow and update pytest events - #68

Merged
hzhangxyz merged 1 commit into
mainfrom
feat-add-jest-workflow
Jan 28, 2026
Merged

ci: add jest workflow and update pytest events#68
hzhangxyz merged 1 commit into
mainfrom
feat-add-jest-workflow

Conversation

@hzhangxyz

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings January 28, 2026 03:36
@hzhangxyz
hzhangxyz merged commit f188c24 into main Jan 28, 2026
30 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds CI coverage for the JavaScript/Jest test suite and slightly adjusts the Pytest workflow trigger ordering.

Changes:

  • Add a new GitHub Actions workflow to run Jest on push and pull requests across a Node.js version matrix.
  • Reorder the Pytest workflow event triggers.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/pytest.yml Reorders workflow triggers for Pytest.
.github/workflows/jest.yml Introduces a Jest CI job running npm ci and npm run all across multiple Node versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 6 to +7
on:
- pull_request
- push
- pull_request

Copilot AI Jan 28, 2026

Copy link

Choose a reason for hiding this comment

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

The only change here is reordering the on: event list. Since YAML list order doesn’t affect the workflow triggers, this creates churn without changing behavior; consider dropping this change unless you’re normalizing formatting across all workflows.

Suggested change
- push
- pull_request
push:
pull_request:

Copilot uses AI. Check for mistakes.
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