XS⚠️ ◾ Refining Pipelines - #857
Conversation
PR Metrics✔ Thanks for keeping your pull request small.
Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs! |
There was a problem hiding this comment.
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.
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
There was a problem hiding this comment.
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
This change refactors the
package.jsonbuild and test scripts to improve efficiency and flexibility. The main change is separatingnpm ci(dependency installation) from the main build, package, and test scripts by introducing new:no-installvariants. 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:
:no-installvariants for build, package, and test scripts to separate dependency installation from the main logic, improving efficiency for repeated runs.build,build:debug,build:release,build:package,test) to callnpm ciexplicitly and delegate to their respective:no-installvariants.Other Improvements:
npm cicalls where not needed, ensuring dependencies are installed only once at the top level.