Skip to content

XS⚠️ ◾ Refining Pipelines - #857

Merged
Muiris Woulfe (muiriswoulfe) merged 2 commits into
mainfrom
muiriswoulfe/pipeline-refinement
Sep 1, 2026
Merged

XS⚠️ ◾ Refining Pipelines#857
Muiris Woulfe (muiriswoulfe) merged 2 commits into
mainfrom
muiriswoulfe/pipeline-refinement

Conversation

@muiriswoulfe

Copy link
Copy Markdown
Member

This change refactors the package.json build and test scripts to improve efficiency and flexibility. The main change is separating npm ci (dependency installation) from the main build, package, and test scripts by introducing new :no-install variants. This allows for faster repeated builds and tests when dependencies haven’t changed, and provides clearer control over when dependencies are installed.

Build and Test Script Refactoring:

  • Added :no-install variants for build, package, and test scripts to separate dependency installation from the main logic, improving efficiency for repeated runs.
  • Updated main scripts (build, build:debug, build:release, build:package, test) to call npm ci explicitly and delegate to their respective :no-install variants.

Other Improvements:

  • Cleaned up initialization scripts by removing redundant npm ci calls where not needed, ensuring dependencies are installed only once at the top level.

@muiriswoulfe
Muiris Woulfe (muiriswoulfe) requested review from a team and a balanced review from Copilot September 1, 2026 11:50
@microsoft-pr-metrics

microsoft-pr-metrics Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

PR Metrics

Thanks for keeping your pull request small.
⚠️ Consider adding additional tests.

Lines
Product Code 35
Test Code -
Subtotal 35
Ignored Code 81
Total 116

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

@microsoft-pr-metrics microsoft-pr-metrics Bot changed the title Refining Pipelines XS⚠️ ◾ Refining Pipelines Sep 1, 2026

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.

Warning

  • Copilot's review of this pull request may be incomplete because some of the changed files are excluded by your Copilot content exclusion settings. See Excluding content from Copilot for details.

Pull request overview

Refactors build and test scripts to install dependencies once, then delegate to reusable :no-install variants.

Changes:

  • Adds no-install build, package, and test commands.
  • Removes redundant installation from initialization scripts.
  • Updates CI/release automation, though those changes were excluded from review.

Reviewed changes

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

Show a summary per file
File Description
package.json Refactors build and test scripts.
.github/workflows/build.yml Content excluded from review.
.github/workflows/release-publish.yml Content excluded from review.
.github/workflows/release-initiate.yml Content excluded from review.
.github/workflow-scripts/New-GeneratedChangesPatch.ps1 Content excluded from review.
Files excluded by content exclusion policy (4)
  • .github/workflow-scripts/New-GeneratedChangesPatch.ps1
  • .github/workflows/build.yml
  • .github/workflows/release-initiate.yml
  • .github/workflows/release-publish.yml

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Super-linter summary

Language Validation result
CHECKOV Pass ✅
EDITORCONFIG Pass ✅
GITHUB_ACTIONS Pass ✅
GITHUB_ACTIONS_ZIZMOR Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
JSCPD Pass ✅
JSON_PRETTIER Pass ✅
MARKDOWN Pass ✅
MARKDOWN_PRETTIER Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
TYPESCRIPT_PRETTIER Pass ✅
XML Pass ✅
YAML Pass ✅
YAML_PRETTIER Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@muiriswoulfe Muiris Woulfe (muiriswoulfe) added the enhancement New feature or request label Sep 1, 2026

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

Warning

  • Copilot's review of this pull request may be incomplete because some of the changed files are excluded by your Copilot content exclusion settings. See Excluding content from Copilot for details.

Pull request overview

Copilot reviewed 1 out of 5 changed files in this pull request and generated no new comments.

Files excluded by content exclusion policy (4)
  • .github/workflow-scripts/New-GeneratedChangesPatch.ps1
  • .github/workflows/build.yml
  • .github/workflows/release-initiate.yml
  • .github/workflows/release-publish.yml

@muiriswoulfe
Muiris Woulfe (muiriswoulfe) merged commit 723c88d into main Sep 1, 2026
37 checks passed
@muiriswoulfe
Muiris Woulfe (muiriswoulfe) deleted the muiriswoulfe/pipeline-refinement branch September 1, 2026 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants