Skip to content

Conversation

@koji
Copy link
Owner

@koji koji commented Jun 3, 2025

No description provided.

@koji koji requested a review from Copilot June 3, 2025 06:08
Copy link

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, titled "update all packages," streamlines the repository by removing legacy configuration files and updating documentation to match a new Storybook React template. Key changes include the removal of the files list (files.ts) and outdated configuration files (e.g., webpack, ESLint, Prettier, Storybook config), plus a comprehensive refresh of the README and license information.

Reviewed Changes

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

Show a summary per file
File Description
files.ts Removed the file list; ensure no tooling relies on these file paths.
README.md Updated documentation to reflect the new Storybook React template and dependency installation flow.
LICENSE Updated licensing header and copyright information.
CHANGELOG.md Removed the changelog; assess if historical change context is maintained elsewhere.
.yarnrc.yml Introduced new Yarn configuration.
.vscode/settings.json Removed VS Code settings.
.storybook/webpack.config.js Removed legacy webpack config in favor of updated Storybook configuration with Vite.
.storybook/preview.ts Added new Storybook preview configuration.
.storybook/main.ts Added new Storybook main configuration.
.storybook/config.js Removed legacy Storybook configuration.
.storybook/addons.js Removed legacy Storybook addons registration.
.prettierrc.js Removed Prettier configuration.
.eslintrc.js Removed old ESLint configuration in favor of a new configuration.
.eslintrc.cjs Added new ESLint configuration.
Comments suppressed due to low confidence (5)

files.ts:1

  • The removal of the files array suggests these paths are no longer needed; please verify that no build scripts or tools rely on this file.
const files = [

CHANGELOG.md:1

  • Removing CHANGELOG.md might hinder users from tracking historical changes; consider preserving it or linking to another detailed changelog medium.
# Changelog

.eslintrc.js:1

  • Since .eslintrc.js has been removed in favor of .eslintrc.cjs, please ensure that all linting scripts and editor integrations are updated accordingly.
module.exports = {

.storybook/config.js:1

  • Ensure that Storybook configuration has been fully migrated to the new main.ts and preview.ts setup so that no legacy config is inadvertently referenced.
import { configure, addDecorator } from "@storybook/react";

.storybook/addons.js:1

  • Verify that the Storybook addons configuration is now handled in the updated setup and that removal of this file will not affect addon registration.
import "@storybook/addon-actions/register";

@koji koji merged commit ff17674 into develop Jun 3, 2025
2 of 6 checks passed
@koji koji deleted the feat_update-all-packages branch June 3, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant