Skip to content

fix: remove unnecessary dependencies#9816

Merged
devongovett merged 3 commits intomainfrom
fix-some-peers
Mar 23, 2026
Merged

fix: remove unnecessary dependencies#9816
devongovett merged 3 commits intomainfrom
fix-some-peers

Conversation

@snowystinger
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

rspbot commented Mar 20, 2026

},
"peerDependencies": {
"@react-spectrum/provider": "^3.9.12",
"@react-spectrum/provider": "^3.10.14",
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.

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

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.

udpated in my followup #9817

@devongovett devongovett added this pull request to the merge queue Mar 23, 2026
Merged via the queue into main with commit ffb1a9d Mar 23, 2026
29 checks passed
@devongovett devongovett deleted the fix-some-peers branch March 23, 2026 19:50
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.

4 participants