Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

feat: add React18 to peerDeps #100

Merged
merged 1 commit into from May 2, 2022
Merged

Conversation

alejandronanez
Copy link
Contributor

@alejandronanez alejandronanez commented Apr 18, 2022

Issue: Probably closes #99

What Changed

Add React18 to the peerDependency list

Checklist

Check the ones applicable to your change:

  • Tests are updated
  • Documentation is updated

Change Type

Indicate the type of change your pull request is:

  • documentation
  • patch
  • minor
  • major

@alejandronanez alejandronanez mentioned this pull request Apr 18, 2022
@yannbf yannbf added the linear add issue to linear label Apr 19, 2022
@@ -78,7 +78,7 @@
"@storybook/client-api": ">=6.4.0",
"@storybook/preview-web": ">=6.4.0",
"@storybook/react": ">=6.4.0",
"react": "^16.8.0 || ^17.0.0"
Copy link
Member

Choose a reason for hiding this comment

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

Yeah this looks like the proper fix to me.

I'd expect a lock-file change associated with this, @yannbf
if for testing purposes you want to stay on a particular version in this repository (whilst support any listed version) it's a good idea to add the exact version you know this repo is ran with as a devDependency too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ndelangen 👋 Just in case, I just ran yarn install locally and I didn't get any changes on my yarn.lock file.

@ndelangen ndelangen merged commit 499a111 into storybookjs:main May 2, 2022
@PeterSHL
Copy link

Hi @ndelangen do you know when is this expected to be released?

@ndelangen
Copy link
Member

Yeah I'd assume so, @alejandronanez ?

@github-actions
Copy link

🚀 PR was released in v1.3.0 🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
linear add issue to linear released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support React@18
4 participants