Skip to content

ci: use pnpm ci to install dependencies#1701

Merged
meteorlxy merged 1 commit into
mainfrom
pnpm-ci
May 14, 2026
Merged

ci: use pnpm ci to install dependencies#1701
meteorlxy merged 1 commit into
mainfrom
pnpm-ci

Conversation

@Mister-Hope
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 14, 2026 07:29
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 25847818580

Coverage remained the same at 73.756%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1034
Covered Lines: 760
Line Coverage: 73.5%
Relevant Branches: 513
Covered Branches: 381
Branch Coverage: 74.27%
Branches in Coverage %: Yes
Coverage Strength: 44.63 hits per line

💛 - Coveralls

Copy link
Copy Markdown

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 updates the repository’s GitHub Actions workflows to use pnpm ci for dependency installation, aiming for a more CI-optimized install step across checks, coverage, e2e, and release pipelines.

Changes:

  • Replace pnpm install --frozen-lockfile with pnpm ci in CI workflows.
  • Apply the same dependency install command consistently across check, coverage, e2e, and release jobs.

Reviewed changes

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

File Description
.github/workflows/release.yml Uses pnpm ci during release workflow dependency installation.
.github/workflows/e2e.yml Uses pnpm ci for e2e workflow dependency installation across OS/node matrix.
.github/workflows/coverage.yml Uses pnpm ci before build and coverage steps.
.github/workflows/check.yml Uses pnpm ci before build/lint/tests in the main check workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 34 to +35
- name: Install dependencies
run: pnpm install --frozen-lockfile
run: pnpm ci
Comment thread .github/workflows/e2e.yml
Comment on lines 39 to +40
- name: Install dependencies
run: pnpm install --frozen-lockfile
run: pnpm ci
Comment on lines 35 to +36
- name: Install dependencies
run: pnpm install --frozen-lockfile
run: pnpm ci
Comment on lines 32 to +33
- name: Install dependencies
run: pnpm install --frozen-lockfile
run: pnpm ci
@meteorlxy meteorlxy changed the title ci: improve pnpm command ci: use pnpm ci to install dependencies May 14, 2026
@meteorlxy meteorlxy merged commit 987f8c1 into main May 14, 2026
33 of 35 checks passed
@meteorlxy meteorlxy deleted the pnpm-ci branch May 14, 2026 08:36
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.

4 participants