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

Fix unit test for Prompty JS #198

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

YusakuNo1
Copy link
Member

Fix TypeScript unit test issue and add unit test for PRs

@YusakuNo1 YusakuNo1 requested review from sethjuarez and Copilot and removed request for sethjuarez February 28, 2025 16:40

Choose a reason for hiding this comment

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

PR Overview

This PR fixes TypeScript unit test issues and adds a unit test workflow for Prompty JS.

  • Introduces a new GitHub Actions workflow to build and test the Prompty JS project on multiple Node.js versions and operating systems.
  • Updates the CI configuration to trigger on pull requests affecting the runtime/promptyjs directory.

Reviewed Changes

File Description
.github/workflows/prompty-js-check.yml Adds a workflow to run unit tests on Node.js versions across OSs.

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

@YusakuNo1 YusakuNo1 merged commit 96d3d5e into microsoft:main Mar 4, 2025
7 checks passed
@YusakuNo1 YusakuNo1 deleted the users/daviwu/fix-js-unit-test branch March 4, 2025 06:11
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.

1 participant