Skip to content

fix(ci): resolve website build failures due to lockfile mismatch#4024

Merged
theianmay merged 4 commits into
react-native-elements:nextfrom
theianmay:fix/ci-lockfile
Jan 27, 2026
Merged

fix(ci): resolve website build failures due to lockfile mismatch#4024
theianmay merged 4 commits into
react-native-elements:nextfrom
theianmay:fix/ci-lockfile

Conversation

@theianmay

Copy link
Copy Markdown
Member

Problem
The website CI build was failing with YN0028: The lockfile would have been modified by this install, which is explicitly forbidden.

This occurred because:

  1. The website depends on rneui/base via file:../packages/base
  2. Yarn Berry automatically enables immutable mode in CI environments
  3. When packages/base changes, the lockfile hash becomes out of sync
  4. yarn install --immutable fails when the hash doesn't match

Solution
Replace yarn install --immutable with yarn install --mode=update-lockfile in the website install step. This flag explicitly allows lockfile updates when workspace dependencies change.

Changes

  • .github/actions/install/action.yml: Update website install command

Additional Improvements

  • Added .gitattributes to enforce consistent LF line endings across platforms
  • Renormalized repository files to use LF line endings

This ensures the website builds successfully in CI while maintaining reproducibility for external npm dependencies.

@codecov

codecov Bot commented Jan 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.23656% with 107 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.40%. Comparing base (4eb18f8) to head (1ef3543).
⚠️ Report is 6 commits behind head on next.

Files with missing lines Patch % Lines
packages/base/src/Slider/Slider.tsx 71.50% 55 Missing ⚠️
packages/base/src/TabView/TabView.tsx 52.54% 28 Missing ⚠️
packages/base/src/Input/Input.tsx 66.66% 13 Missing ⚠️
packages/base/src/Tooltip/Tooltip.tsx 84.72% 11 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             next    #4024   +/-   ##
=======================================
  Coverage   79.40%   79.40%           
=======================================
  Files          90       90           
  Lines        2141     2141           
  Branches      954      919   -35     
=======================================
  Hits         1700     1700           
  Misses        439      439           
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@theianmay theianmay changed the title fix(website): Build Failure Due to Lockfile Hash Mismatch fix(ci): resolve website build failures due to lockfile mismatch Jan 27, 2026
@theianmay
theianmay marked this pull request as ready for review January 27, 2026 06:20
@theianmay
theianmay merged commit 7661c64 into react-native-elements:next Jan 27, 2026
8 of 9 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress 🚧 to Closed or Merged 🏁 in Community Roadmap 🛣 Jan 27, 2026
github-actions Bot pushed a commit that referenced this pull request Jan 27, 2026
fix(ci): resolve website build failures due to lockfile mismatch
github-actions Bot pushed a commit that referenced this pull request Jan 27, 2026
fix(ci): resolve website build failures due to lockfile mismatch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Closed or Merged 🏁

Development

Successfully merging this pull request may close these issues.

1 participant