Skip to content

fix ci error#280

Merged
eve-git merged 1 commit into
bcgov:mainfrom
eve-git:fix-ci-error
Jun 17, 2026
Merged

fix ci error#280
eve-git merged 1 commit into
bcgov:mainfrom
eve-git:fix-ci-error

Conversation

@eve-git

@eve-git eve-git commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@eve-git

eve-git commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@bcregistry-sre

Copy link
Copy Markdown
Collaborator

Temporary Url for review: https://documents-ui-dev--pr-280-uuk0ce9z.web.app

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.

Pull request overview

This PR aims to fix CI instability/errors for documents-ui by making dependency resolution deterministic under pnpm.

Changes:

  • Pinned several Nuxt-related dependencies (and test tooling) to exact versions to prevent unexpected upgrades.
  • Added an explicit eslint devDependency and pinned Vitest-related versions.
  • Introduced pnpm.onlyBuiltDependencies and pnpm.overrides to control native build steps and force specific transitive versions.

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

@eve-git eve-git merged commit 400d423 into bcgov:main Jun 17, 2026
8 checks passed
@eve-git eve-git deleted the fix-ci-error branch June 17, 2026 22:08
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