Skip to content

fix: ensure fonts are included in release source package#1663

Open
hobostay wants to merge 1 commit intoTencent:mainfrom
hobostay:fix/missing-fonts-in-release
Open

fix: ensure fonts are included in release source package#1663
hobostay wants to merge 1 commit intoTencent:mainfrom
hobostay:fix/missing-fonts-in-release

Conversation

@hobostay
Copy link
Copy Markdown

Summary

  • 修改 gulpfile.js,让字体文件同时输出到 src/sass/fonts/dist/fonts/
  • src/sass/fonts/ 目录添加到 git,确保 release 源码包包含字体文件

Related Issue

Fixes #1651

Problem

release 源码包中缺少 src/sass/fonts/ 目录,导致用户下载后运行 yarn dev 时图标丢失。

Solution

  1. 修改 gulpfile.js 的 handleFonts 函数,同时输出字体到两个位置:

    • src/sass/fonts/ - 用于开发环境
    • dist/fonts/ - 用于生产环境
  2. 将生成的字体文件添加到版本控制,确保 release 源码包包含它们

Test plan

  • 本地运行 yarn iconfont 生成字体文件
  • 验证字体文件存在于 src/sass/fonts/ 目录
  • 代码审查

🤖 Generated with Claude Code

Fixes Tencent#1651

问题:release 源码包中缺少 src/sass/fonts/ 目录,导致 yarn dev 时图标丢失

解决方案:
1. 修改 gulpfile.js,让字体文件同时输出到 src/sass/fonts/ 和 dist/fonts/
2. 将 src/sass/fonts/ 目录添加到 git,确保 release 源码包包含字体文件

Changes:
- gulpfile.js: 同时输出字体到 src/sass/fonts/ 和 dist/fonts/
- src/sass/fonts/: 添加字体文件到版本控制

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 29, 2026

⚠️ No Changeset found

Latest commit: d2207a5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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