Skip to content

fix: clean up deprecated config and strengthen CI#463

Merged
Leaf-0w0 merged 5 commits into
LyraVoid:masterfrom
sleep1223:codex/type-check-deprecation-cleanup
May 13, 2026
Merged

fix: clean up deprecated config and strengthen CI#463
Leaf-0w0 merged 5 commits into
LyraVoid:masterfrom
sleep1223:codex/type-check-deprecation-cleanup

Conversation

@sleep1223
Copy link
Copy Markdown
Contributor

@sleep1223 sleep1223 commented May 13, 2026

Type of change

  • Bug fix (a non-breaking change that fixes an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (please describe): CI 覆盖补充与弃用项清理

Checklist

  • I have read the CONTRIBUTING document.
  • I have checked to ensure that this Pull Request is not for personal changes.
  • I have performed a self-review of my own code.
  • My changes generate no new warnings.

Related Issue

N/A

Changes

  • 移除 TypeScript 已弃用的 baseUrl 选项,并将路径别名更新为显式的 ./src/... 目标路径。
  • pnpm type-check 更新为运行 tsc --noEmit,并补充 Astro 与 Svelte 组件模块声明,使独立 TypeScript 检查能够覆盖这些导入。
  • 在 CI workflow 中新增 pnpm type-check,确保 TypeScript 配置类问题能在合并前被发现。
  • 将已弃用的 Svelte on:event 指令迁移为事件属性写法。
  • 移除已弃用的 document.execCommand("copy") 剪贴板备用逻辑,改为依赖 Clipboard API。
  • 通过禁用 ttf2woff2 额外 clone 步骤,避免字体压缩过程中触发 Fontmin 的弃用文件 clone 路径。

How To Test

  • nr type-check
  • nr check
  • nr build
  • node --trace-deprecation scripts/compress-fonts.js

Screenshots (if applicable)

N/A. This PR does not include visual UI changes.

Additional Notes

本次改动源于 TypeScript 6 对 baseUrl 的弃用警告。此前 GitHub Actions 没有运行独立的 pnpm type-check,因此该问题没有在 CI 中暴露。

pnpm build 仍可能输出项目中既有的 Vite 警告,例如无关源码里的未使用导入;这些警告并非本 PR 新增。

@sleep1223 sleep1223 changed the title Codex/type check deprecation cleanup fix: clean up deprecated config and strengthen CI May 13, 2026
@sleep1223 sleep1223 marked this pull request as ready for review May 13, 2026 08:00
@Leaf-0w0 Leaf-0w0 merged commit 680a230 into LyraVoid:master May 13, 2026
4 checks passed
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