-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Core: Wrap manager entries to handle exports using a cache directory #20331
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Still not 100% confident, but let's see what users say
…js/storybook into fix/manager-entry-with-exports
I make this mistake all the time myself, but it's a bit risky to use types from before a breaking change. {
"@types/find-cache-dir": "^3.2.1",
"find-cache-dir": "^4.0.0",
} Maybe a test to check for mismatching types would be prudent? Anyways, this may be a pointer:
Wishing you the best of luck, and I'm looking forward to have this solved to get back on the newest beta version 🙏 |
Thanks for spotting that @VidunderGunder ❤️ I've opened a PR downgrading for now. |
Issue: managerEntries can break when they contains exports
What I did
see: #20308
This would also allow us to provide cleaner locations without special characters in the paths for managerEntries.