Skip to content

chore(ci): ignore react package-lock changes for nightly - #31542

Merged
OS-jacobbell merged 1 commit into
mainfrom
fix/nightly
Oct 7, 2026
Merged

OS-jacobbell merged 1 commit into
mainfrom
fix/nightly

Conversation

@OS-jacobbell

Copy link
Copy Markdown
Contributor

What is the current behavior?

Stencil Nightly Build installs the latest @stencil/react-output-target, which changes packages/react/package-lock.json. When it checks for a clean git diff, the altered package-lock file fails CI.

What is the new behavior?

  • Clean up packages/react/package-lock.json along with package.json.

Does this introduce a breaking change?

  • Yes
  • No

@OS-jacobbell
OS-jacobbell requested a review from a team as a code owner October 7, 2026 16:34
@OS-jacobbell
OS-jacobbell requested a review from ShaneK October 7, 2026 16:34
@vercel

vercel Bot commented Oct 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ionic-framework Ready Ready Preview Oct 7, 2026 4:34pm UTC

Request Review

@OS-jacobbell OS-jacobbell changed the title chore(ci): ignore package-lock changes chore(ci): ignore packages/react/package-lock.json changes Oct 7, 2026
@OS-jacobbell OS-jacobbell changed the title chore(ci): ignore packages/react/package-lock.json changes chore(ci): ignore react package-lock changes for nightly Oct 7, 2026

@thetaPC thetaPC 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.

LGTM

@OS-jacobbell
OS-jacobbell added this pull request to the merge queue Oct 7, 2026
Merged via the queue into main with commit f7b5988 Oct 7, 2026
106 checks passed
@OS-jacobbell
OS-jacobbell deleted the fix/nightly branch October 7, 2026 18:29

This branch was successfully deployed

1 active deployment
Preview — 164bdd78 Deployed Oct 7, 2026 by vercel[bot]
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.

2 participants