Skip to content

chore: Lockfiles for all apps#8940

Merged
snowystinger merged 32 commits into
mainfrom
lockfiles-for-all-apps
Oct 8, 2025
Merged

chore: Lockfiles for all apps#8940
snowystinger merged 32 commits into
mainfrom
lockfiles-for-all-apps

Conversation

@snowystinger

@snowystinger snowystinger commented Sep 29, 2025

Copy link
Copy Markdown
Member

Closes

Split out from #8913
This just focuses on lock files and install states for all of our examples and starters. It's the first PR because it's probably the most important to establish. From here we can see what future changes are actually doing and something new can't come in and cause a different issue. Think of it as establishing a baseline.

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

Comment thread Makefile

starter:
cd starters/docs && yarn --no-immutable && yarn tsc
cd starters/docs && yarn --no-immutable && yarn up react-aria-components && yarn tsc

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this forces the update from the version provided by the root workspace, to the latest tag on verdaccio. I'm not sure why there is no way to override this yarn behaviour and not look at the root workspace at all, but this appears to be the case currently. I think due to yarnpkg/yarn#4099

@rspbot

rspbot commented Sep 29, 2025

Copy link
Copy Markdown

@rspbot

rspbot commented Sep 29, 2025

Copy link
Copy Markdown

Comment thread examples/rsp-cra-18/package.json
@rspbot

rspbot commented Oct 3, 2025

Copy link
Copy Markdown

@rspbot

rspbot commented Oct 7, 2025

Copy link
Copy Markdown

@rspbot

rspbot commented Oct 7, 2025

Copy link
Copy Markdown

LFDanLu
LFDanLu previously approved these changes Oct 8, 2025

@LFDanLu LFDanLu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified that the proper bumped version is being published via verdaccio, will double check the installed lockfile when I get back home/tmrw but should be fine

@rspbot

rspbot commented Oct 8, 2025

Copy link
Copy Markdown

@snowystinger snowystinger added this pull request to the merge queue Oct 8, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 8, 2025
@snowystinger snowystinger enabled auto-merge October 8, 2025 23:14
@rspbot

rspbot commented Oct 8, 2025

Copy link
Copy Markdown

@snowystinger snowystinger added this pull request to the merge queue Oct 8, 2025
Merged via the queue into main with commit ee53a79 Oct 8, 2025
32 checks passed
@snowystinger snowystinger deleted the lockfiles-for-all-apps branch October 8, 2025 23:25
devongovett pushed a commit that referenced this pull request Oct 9, 2025
* chore: lockfile for all apps

* turn on verdaccio

* set hoisting limit

* build starters outside root workspace

* fix indentation

* force dependency update to latest

* try another command

* try yet another command

* fix paths

* fix more paths

* reduce complexity and force upgrade locally

* Revert "turn on verdaccio"

This reverts commit 5bff9d5.

* force update all versions in all example apps

* turn on verdaccio

* fix webpack 4 app

* fix icon builder

* try cache cleaning

* force npx to use local verdaccio registry

* try another method of forcing the registry

* specify a specific version for the tool

* debugging

* remove debug statement

* Revert "turn on verdaccio"

This reverts commit 37e6c89.

* turn on verdaccio

* Revert "turn on verdaccio"

This reverts commit ca2faf9.

* turn on verdaccio

* add debugging

* one more debug line

* Revert "turn on verdaccio"

This reverts commit 622ad5e.
bsmiley1 pushed a commit to bsmiley1/react-spectrum that referenced this pull request Dec 5, 2025
* chore: lockfile for all apps

* turn on verdaccio

* set hoisting limit

* build starters outside root workspace

* fix indentation

* force dependency update to latest

* try another command

* try yet another command

* fix paths

* fix more paths

* reduce complexity and force upgrade locally

* Revert "turn on verdaccio"

This reverts commit 5bff9d5.

* force update all versions in all example apps

* turn on verdaccio

* fix webpack 4 app

* fix icon builder

* try cache cleaning

* force npx to use local verdaccio registry

* try another method of forcing the registry

* specify a specific version for the tool

* debugging

* remove debug statement

* Revert "turn on verdaccio"

This reverts commit 37e6c89.

* turn on verdaccio

* Revert "turn on verdaccio"

This reverts commit ca2faf9.

* turn on verdaccio

* add debugging

* one more debug line

* Revert "turn on verdaccio"

This reverts commit 622ad5e.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants