Skip to content

fix: remove unnecessary dependencies#9816

Open
snowystinger wants to merge 3 commits intomainfrom
fix-some-peers
Open

fix: remove unnecessary dependencies#9816
snowystinger wants to merge 3 commits intomainfrom
fix-some-peers

Conversation

@snowystinger
Copy link
Member

Closes

Fixes a number of these
Screenshot 2026-03-20 at 1 12 21 pm

✅ 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:

@github-actions github-actions bot added the v3 label Mar 20, 2026
@rspbot
Copy link

rspbot commented Mar 20, 2026

"devDependencies": {
"@adobe/spectrum-css-temp": "3.0.0-alpha.1",
"@react-spectrum/test-utils-internal": "3.0.0-alpha.1"
"@adobe/spectrum-css-temp": "3.0.0-alpha.1"
Copy link
Member

Choose a reason for hiding this comment

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

probably don't need this one anymore either

Copy link
Member Author

@snowystinger snowystinger Mar 20, 2026

Choose a reason for hiding this comment

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

yeah, I was mulling that one over, I'll go ahead and remove it
hooray for simplifying our constraints!

Copy link
Member Author

@snowystinger snowystinger Mar 20, 2026

Choose a reason for hiding this comment

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

probably don't even need the dependency on react for them either....
eh, then i think they'll get made that they aren't providing the peer that the mono package requires, nevermind

devongovett
devongovett previously approved these changes Mar 20, 2026
@rspbot
Copy link

rspbot commented Mar 20, 2026

},
"peerDependencies": {
"@react-spectrum/provider": "^3.9.12",
"@react-spectrum/provider": "^3.10.14",
Copy link
Member

Choose a reason for hiding this comment

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

We can do this separately I guess, but test utils should depend on the monopackage instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants