fix: size report workflow

[Rachit1313]

Description:

This pull request addresses a TypeScript type error in the vite.config.size.ts file. The issue was related to the type inference of the base configuration. TypeScript was expecting a function call, but base was a function itself.

To resolve this, I explicitly annotated the type of the base configuration in vite.config.size.ts file. Additionally, I ensured that the base function is called with the required parameter { mode: process.env.NODE_ENV } in the vite.config.size.ts file.

These changes fix the TypeScript type errors and allow for a smooth build process.

Changes Made:

• Modified vite.config.size.ts to call the base function with the necessary parameter.

Testing:

• Tested the changes locally to ensure there are no TypeScript type errors.
• Verified that the build process completes successfully after the modifications.

This resolves issue #5408
close #5408

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
blocksuite	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 23, 2023 7:21am

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
blocksuite-docs	⬜️ Ignored (Inspect)	Visit Preview		Nov 23, 2023 7:21am

[affine-bot]

Hello! Thank you for your contribution.

In order for us to review and merge your suggested changes, please sign our CLA at https://github.com/toeverything/blocksuite/edit/master/.github/CLA.md

[Rachit1313]

This commit passed all the tests on the forked repo.

[Rachit1313]

@doouding / @doodlewind , Could you please guide with the PR title lint and regarding the test. It passed in my forked repo, Is that possible for you to re run the failed jobs?

[Rachit1313]

@doodlewind , The PR title lint still failed

[Rachit1313]

@doodlewind , The PR title lint still failed

Except the PR title lint, everything passed

[Rachit1313]

@doodlewind , Can you link issue #5408

[doodlewind]

@Rachit1313 I think you can simply add "close #xxx" in your PR description.

[Rachit1313]

@Rachit1313 I think you can simply add "close #xxx" in your PR description.

Done, Thanks. Every check is passing except PR title lint, not sure why?

[Rachit1313]

failed with ✖ scope must be one of [page, edgeless, database, virgo, store, std, playground]

[Rachit1313]

failed with ✖ scope must be one of [page, edgeless, database, virgo, store, std, playground]

@doodlewind Based on your lint file, ["page", "edgeless", "database", "virgo", "store", "std", "playground"] These are the only scopes supported and I don't see ci or cu here.

[Rachit1313]

@doodlewind , good to go. Let me know if any changes are required before merging?

[doodlewind]

LGTM, thanks!