Skip to content

Updated dependencies#2

Merged
dew326 merged 5 commits intomainfrom
update-deps
Jun 9, 2025
Merged

Updated dependencies#2
dew326 merged 5 commits intomainfrom
update-deps

Conversation

@GrabowskiM
Copy link
Copy Markdown
Contributor

🎫 Issue IBX-XXXXX

Description:

For QA:

Documentation:

@GrabowskiM GrabowskiM requested a review from Copilot June 5, 2025 10:58
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates project dependencies and configuration settings to align with the latest versions, including node and Storybook changes.

  • Updated import paths in story files to reference the new 'storybook/test' module.
  • Upgraded various package versions in package.json.
  • Removed unused Storybook addons and updated node versions in CI workflows.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/components/src/Expander/Expander.test.stories.tsx Changed story import path to the updated module.
packages/components/src/Button/Button.test.stories.tsx Changed story import path to the updated module.
packages/components/src/Accordion/Accordion.test.stories.tsx Changed story import path to the updated module.
package.json Upgraded dependencies and devDependencies for consistency.
.storybook/main.js Removed deprecated addons for improved configuration.
.storybook/decorators.tsx Updated ContextDecorator signature with a looser type.
.github/workflows/storybook-tests.yaml Updated Node version for CI consistency.
.github/workflows/frontend-ci.yaml Updated Node version for CI consistency.
Comments suppressed due to low confidence (4)

.storybook/decorators.tsx:25

  • [nitpick] Consider replacing 'any' with the appropriate Story type from '@storybook/react' to improve type safety instead of disabling lint rules.
export const ContextDecorator = (Story: any, // eslint-disable-line @typescript-eslint/no-explicit-any

packages/components/src/Expander/Expander.test.stories.tsx:2

  • Verify that the updated module path 'storybook/test' is correct and resolves properly with the new dependency configuration.
import { expect, fn, userEvent, within } from 'storybook/test';

packages/components/src/Button/Button.test.stories.tsx:2

  • Verify that the updated module path 'storybook/test' is correct and resolves properly with the new dependency configuration.
import { expect, fn, userEvent, within } from 'storybook/test';

packages/components/src/Accordion/Accordion.test.stories.tsx:4

  • Verify that the updated module path 'storybook/test' is correct and resolves properly with the new dependency configuration.
import { expect, fn, userEvent, within } from 'storybook/test';

@GrabowskiM GrabowskiM requested a review from a team June 5, 2025 11:21
@ezrobot ezrobot requested review from OstafinL, RopRaptor, albozek, dew326 and tischsoic and removed request for a team June 5, 2025 11:21
@GrabowskiM GrabowskiM marked this pull request as draft June 5, 2025 11:41
@GrabowskiM GrabowskiM marked this pull request as ready for review June 6, 2025 07:34
@dew326 dew326 merged commit 121fa48 into main Jun 9, 2025
4 checks passed
@dew326 dew326 deleted the update-deps branch June 9, 2025 09:03
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.

5 participants