Skip to content

Conversation

mbg
Copy link
Member

@mbg mbg commented Sep 25, 2025

A few minor bits bundled up:

  • Adds a npm run ava command; which can be used to run tests based on a custom path.
  • Updates npm run test to use that, with the existing src/ path.
  • Updates CONTRIBUTING.md to highlight npm run ava
  • Updates the justfile to use npm run ava
  • Filters transpiled files and node_modules from search results in VSCode

Risk assessment

For internal use only. Please select the risk level of this change:

  • Low risk: Changes are fully under feature flags, or have been fully tested and validated in pre-production environments and are highly observable, or are documentation or test only.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Consider adding a changelog entry for this change.
  • Confirm the readme and docs have been updated if necessary.

@mbg mbg requested a review from a team as a code owner September 25, 2025 11:48
@Copilot Copilot AI review requested due to automatic review settings September 25, 2025 11:48
Copy link
Contributor

@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 adds development tooling improvements to streamline test execution and improve the development experience. It introduces a flexible npm run ava command for running tests on custom paths, while maintaining backward compatibility with the existing test workflow.

  • Adds a new npm run ava script to enable flexible test path specification
  • Updates existing npm run test to use the new script with default src/ path
  • Updates documentation and justfile to reference the new command
  • Excludes transpiled files from VSCode search results

Reviewed Changes

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

Show a summary per file
File Description
package.json Adds ava script and updates test script to use it with default path
lib/*.js Auto-generated JavaScript files reflecting package.json changes
justfile Updates test_file recipe to use new npm script
CONTRIBUTING.md Documents the new testing command for specific paths
.vscode/settings.json Excludes build artifacts from search results

@mbg mbg force-pushed the mbg/node/individual-test-cmd branch 2 times, most recently from 01f46d9 to 4ebd49f Compare September 25, 2025 12:25
@mbg mbg force-pushed the mbg/node/individual-test-cmd branch from 4ebd49f to 48be21c Compare September 25, 2025 13:22
@mbg mbg requested a review from henrymercer September 25, 2025 13:29
@mbg mbg merged commit 853decd into main Sep 25, 2025
238 of 239 checks passed
@mbg mbg deleted the mbg/node/individual-test-cmd branch September 25, 2025 13:42
@github-actions github-actions bot mentioned this pull request Sep 26, 2025
8 tasks
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