Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: full use vite to build pkg #1853

Merged
merged 6 commits into from
May 12, 2024
Merged

build: full use vite to build pkg #1853

merged 6 commits into from
May 12, 2024

Conversation

Rhilip
Copy link
Collaborator

@Rhilip Rhilip commented May 8, 2024

移除webpack构建部分,全部转为vite构建。

普通用户请勿在生产环境中使用本pr的相关action构建。

@Rhilip Rhilip requested a review from ted423 May 8, 2024 15:29
@Rhilip
Copy link
Collaborator Author

Rhilip commented May 8, 2024

@trim21 请一并review(圈不出来2333

@trim21
Copy link

trim21 commented May 8, 2024

  1. esbuild-loadervuetify-loader 两个loader没删
  2. 建议把 build.chunkSizeWarningLimit 设置成 Number.MAX_SAFE_INTEGER,设置成0依旧会警告。。。

其他的好像没啥问题

同时增加 MiniDB 标签
@ronggang
Copy link
Collaborator

拉回来构建试了一下,改成vite后构建速度要快很多。 👍

@Rhilip Rhilip marked this pull request as ready for review May 10, 2024 14:02
@Rhilip Rhilip merged commit 38a99e4 into dev May 12, 2024
4 checks passed
@Rhilip Rhilip deleted the dev_full_vite branch May 12, 2024 01:42
@fzlins
Copy link
Collaborator

fzlins commented May 14, 2024

#1857 (comment)

dayjs无法正常声明了。导致获取加入时间出错,后续逻辑跳过。

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.

4 participants