Skip to content

Fix escaping in npm scripts in package.json#7771

Merged
hardillb merged 1 commit into
mainfrom
7770-fix-npm-test-scripts-for-windows
Jul 7, 2026
Merged

Fix escaping in npm scripts in package.json#7771
hardillb merged 1 commit into
mainfrom
7770-fix-npm-test-scripts-for-windows

Conversation

@Steve-Mcl

Copy link
Copy Markdown
Contributor

Description

This pull request makes minor updates to the package.json scripts for consistency. The main change is switching single quotes to double quotes around glob patterns and paths in several npm scripts. This helps ensure cross-platform compatibility, especially for Windows environments.

Most important changes:

Script command consistency and compatibility:

  • Changed single quotes to double quotes in the test:unit:forge and test:system script glob patterns to improve compatibility across different shells and operating systems.

Related Issue(s)

closes #7770

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

@Steve-Mcl
Steve-Mcl requested a review from knolleary July 7, 2026 15:50
@Steve-Mcl Steve-Mcl linked an issue Jul 7, 2026 that may be closed by this pull request
@hardillb
hardillb enabled auto-merge (squash) July 7, 2026 16:24
@hardillb
hardillb merged commit 8af93d7 into main Jul 7, 2026
35 of 37 checks passed
@hardillb
hardillb deleted the 7770-fix-npm-test-scripts-for-windows branch July 7, 2026 16:26
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.

NPM test scripts do not work on windows

2 participants