Skip to content

Build: Update dependencies in yarn.lock and clean up comments#33089

Merged
yannbf merged 1 commit into
nextfrom
norbert/fixing-linting-on-next
Nov 19, 2025
Merged

Build: Update dependencies in yarn.lock and clean up comments#33089
yannbf merged 1 commit into
nextfrom
norbert/fixing-linting-on-next

Conversation

@ndelangen
Copy link
Copy Markdown
Member

@ndelangen ndelangen commented Nov 19, 2025

What I did

I'm hoping to address this:
https://app.circleci.com/pipelines/github/storybookjs/storybook/108653/workflows/ea4dedbc-1ce4-49fc-a444-ac1f27c94a84/jobs/948062

I suspect it's due to some dependencies having been updated in code, but not in scripts, so I'm doing a complete regeneration of the lockfiles

Regenerating the scripts lockfile, causes out of memory issues whilst generating dts bundles!
So I had to be selective there.

Checklist for Contributors

Testing

The changes in this PR are covered in the following automated tests:

  • stories
  • unit tests
  • integration tests
  • end-to-end tests

Manual testing

No specific manual testing required.

Documentation

  • Add or update documentation reflecting your changes
  • If you are deprecating/removing a feature, make sure to update
    MIGRATION.MD

Checklist for Maintainers

  • When this PR is ready for testing, make sure to add ci:normal, ci:merged or ci:daily GH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found in code/lib/cli-storybook/src/sandbox-templates.ts

  • Make sure this PR contains one of the labels below:

    Available labels
    • bug: Internal changes that fixes incorrect behavior.
    • maintenance: User-facing maintenance tasks.
    • dependencies: Upgrading (sometimes downgrading) dependencies.
    • build: Internal-facing build tooling & test updates. Will not show up in release changelog.
    • cleanup: Minor cleanup style change. Will not show up in release changelog.
    • documentation: Documentation only changes. Will not show up in release changelog.
    • feature request: Introducing a new feature.
    • BREAKING CHANGE: Changes that break compatibility in some way with current major version.
    • other: Changes that don't fit in the above categories.

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

core team members can create a canary release here or locally with gh workflow run --repo storybookjs/storybook publish.yml --field pr=<PR_NUMBER>

- Updated various package versions in yarn.lock, including @chromatic-com/storybook, @emnapi/core, and rollup.
- Cleaned up comments in dangerfile.js, npm-auth.ts, codemod.ts, sandbox-parts.ts, main-js.ts, and tools.ts to improve readability and maintainability.
@ndelangen ndelangen requested a review from yannbf November 19, 2025 10:44
@ndelangen ndelangen self-assigned this Nov 19, 2025
@ndelangen ndelangen added dependencies ci:merged Run the CI jobs that normally run when merged. labels Nov 19, 2025
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Nov 19, 2025

View your CI Pipeline Execution ↗ for commit 2a5948b

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 44s View ↗

☁️ Nx Cloud last updated this comment at 2025-11-19 11:03:33 UTC

@yannbf yannbf changed the title Update dependencies in yarn.lock and clean up comments in scripts Build: Update dependencies in yarn.lock and clean up comments Nov 19, 2025
@yannbf yannbf merged commit ed1de60 into next Nov 19, 2025
83 of 87 checks passed
@yannbf yannbf deleted the norbert/fixing-linting-on-next branch November 19, 2025 11:21
@github-actions github-actions Bot mentioned this pull request Nov 19, 2025
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:merged Run the CI jobs that normally run when merged. dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants