Skip to content

Fix/audit findings - #14

Merged
MaherJon merged 2 commits into
mainfrom
fix/audit-findings
Oct 18, 2026
Merged

MaherJon merged 2 commits into
mainfrom
fix/audit-findings

Conversation

@MaherJon

@MaherJon MaherJon commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

📝 描述

Fixes #(issue)


🧪 测试

  • 本地构建成功
  • 编译测试通过
  • 功能测试

📋 检查清单

  • 代码符合项目风格
  • 没有 console.log (调试代码)
  • 没有 TODO / FIXME
  • 文档已更新 (如需要)
  • 测试已添加 (如需要)

⚠️ 已知问题

git -C core 会切换工作目录,导致相对路径 patches/core/*.patch 被解析为
core/patches/...,检查时报 "can't open patch: No such file or directory"。
改为传绝对路径后补丁可被正常读取与校验。
android-actions/setup-android@v3 会执行 `sdkmanager tools`,而新版 SDK
仓库已移除 tools 包,导致 setup 步骤报 "Failed to find package 'tools'"
并 exit 1(v4.0.2 的官方修复即 "Fix for removed tools package")。
@MaherJon
MaherJon merged commit 44fe7b0 into main Oct 18, 2026
6 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.

1 participant