Skip to content

[Refactor] Publish workflow check#9

Merged
wzh1994 merged 122 commits intowzh1994:wzh/cpp_initfrom
CompromisedKiwi:yzh/publish_workflow_check
Dec 23, 2025
Merged

[Refactor] Publish workflow check#9
wzh1994 merged 122 commits intowzh1994:wzh/cpp_initfrom
CompromisedKiwi:yzh/publish_workflow_check

Conversation

@CompromisedKiwi
Copy link

📌 PR 内容 / PR Description

  • 切换构建后端到scikit

🔍 相关 Issue / Related Issue

✅ 变更类型 / Type of Change

  • 修复 Bug / Bug fix (non-breaking change that fixes an issue)
  • 新功能 / New feature (non-breaking change that adds functionality)
  • 重构 / Refactor (no functionality change, code structure optimized)
  • 重大变更 / Breaking change (fix or feature that would cause existing functionality to change)
  • 文档更新 / Documentation update (changes to docs only)
  • 性能优化 / Performance optimization

🧪 如何测试 / How Has This Been Tested?

📷 截图 / Demo (Optional)

⚡ 更新后的用法示例 / Usage After Update

# 示例 / Example

🔄 重构前 / 重构后对比 (仅当 Type 为 Refactor) / Refactor Before & After (only for Refactor)

重构前 / Before:

重构后 / After:

⚠️ 注意事项 / Additional Notes

# for test executables
set(LAZYLLM_EXE_FILES
)
# set(LAZYLLM_EXE_FILES)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个留着即可,后面用作单元测试


pattern = re.compile(r"""
^\s*
(?P<name>[A-Za-z0-9_.-]+) # 包名
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

去掉中文注释,下同


pattern = re.compile(r"""
^\s*
(?P<name>[A-Za-z0-9_.-]+) # 包名
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

去掉中文注释,下同

@wzh1994 wzh1994 merged commit feccb4c into wzh1994:wzh/cpp_init Dec 23, 2025
@CompromisedKiwi CompromisedKiwi deleted the yzh/publish_workflow_check branch December 30, 2025 01:51
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