Add npm run test step to README.md workflows for local testing before committing#1244
Add npm run test step to README.md workflows for local testing before committing#1244
Conversation
|
@jasonjoh 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review. |
Co-authored-by: jasonjoh <8966342+jasonjoh@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
This PR adds testing instructions to the README.md workflows to encourage contributors to validate their changes locally before committing. The change addresses issue #107 by adding npm run test steps to both the main workflow and command line workflow sections.
- Added testing step to the main numbered workflow
- Added new testing section to command line workflow with proper renumbering
- Maintains workflow sequence by updating subsequent section numbers
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Updated the README.md to include testing instructions in both sample query workflows, ensuring contributors validate their changes locally before committing.
Changes Made
Main workflow section:
npm run testto test your changes locally" as step 3 in the numbered workflowCommand line workflow section:
npm run testBenefits
This change helps contributors:
The existing test infrastructure validates critical aspects like unique GUIDs, proper URL formats, valid JSON in POST bodies, and other sample query requirements.
Fixes #107.
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
docs.microsoft.comnode scripts/test-initiator.js --no-watch(dns block)learn.microsoft.comnode scripts/test-initiator.js --no-watch(dns block)If you need me to access, download, or install something from one of these locations, you can either:
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.