Skip to content

Conversation

@psd-coder
Copy link
Owner

This pull request introduces a workflow to check the size of specific exports in the project and ensures that size limits are enforced. It adds a new GitHub Actions workflow, updates dependencies, and configures size-limit checks for key exports.

@psd-coder psd-coder self-assigned this May 6, 2025
@psd-coder psd-coder force-pushed the setup-size-limit branch 2 times, most recently from d63b93d to 54b03b2 Compare May 6, 2025 21:28
@psd-coder psd-coder requested a review from Copilot May 6, 2025 21:29
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 pull request introduces a new mechanism to monitor the bundle size of key exports by adding a GitHub Actions workflow, a size-limit configuration file, and updating the README badge.

  • Added bundle size badge to the README to reflect export sizes
  • Configured export size limits via a new .size-limit.ts file
  • Integrated a GitHub Actions workflow to enforce size restrictions

Reviewed Changes

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

File Description
README.md Added a bundle size badge to display current bundle size
.size-limit.ts Configured size limits for key exports using size-limit configuration
.github/workflows/size-limit.yml Added a GitHub Actions workflow to run size-limit checks
Files not reviewed (2)
  • package.json: Language not supported
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (2)

.github/workflows/size-limit.yml:1

  • [nitpick] The workflow name 'Check types' might be misleading since the workflow is intended to enforce size limits. Consider renaming it to something like 'Check bundle size' for clarity.
name: Check types

.github/workflows/size-limit.yml:5

  • [nitpick] Verify that 'master' is the intended branch for triggering the workflow; if your repository uses 'main' as the default branch, update this branch name accordingly.
      - master

@psd-coder psd-coder force-pushed the setup-size-limit branch 3 times, most recently from 41bd96a to fa3045d Compare May 6, 2025 21:45
@psd-coder psd-coder force-pushed the setup-size-limit branch from fa3045d to f53a451 Compare May 6, 2025 21:47
@psd-coder psd-coder merged commit d50376c into main May 6, 2025
3 checks passed
@psd-coder psd-coder deleted the setup-size-limit branch May 6, 2025 21:49
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.

2 participants