Skip to content
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

Dependencies: Upgrade file-system-cache #24232

Merged
merged 8 commits into from
Sep 19, 2023
Merged

Dependencies: Upgrade file-system-cache #24232

merged 8 commits into from
Sep 19, 2023

Conversation

seriouz
Copy link
Contributor

@seriouz seriouz commented Sep 19, 2023

See the old PR #23253

Reverts #23221 as the issue has now been fixed upstream.

What I did

Removed the pinned dependency file-system-cache with 2.3.0 and set it to the fixed version 2.4.1.
Links:

This is the result of yarn why file-system-cache in code:

├─ @storybook/core-common@workspace:lib/core-common
│  └─ file-system-cache@npm:2.4.1 (via npm:^2.4.1)
│
└─ @storybook/types@workspace:lib/types
   └─ file-system-cache@npm:2.4.1 (via npm:^2.4.1)

How to test

Must be tested in a case-sensitive operating system like Linux - not MacOS or Windows.

Checklist

  • Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests)
  • Make sure to add/update documentation regarding your changes
  • If you are deprecating/removing a feature, make sure to update
    MIGRATION.MD

Maintainers

  • If this PR should be tested against many or all sandboxes,
    make sure to add the ci:merged or ci:daily GH label to it.
  • Make sure this PR contains one of the labels below.

["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

@ndelangen ndelangen self-assigned this Sep 19, 2023
@ndelangen ndelangen changed the title Dependencies: Upgrade file-system-cache Dependencies: Upgrade file-system-cache Sep 19, 2023
@ndelangen ndelangen merged commit cf5519b into storybookjs:next Sep 19, 2023
14 of 15 checks passed
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.

3 participants