Star75
18
代码
代码
Issues1
Pull Requests3
项目讨论
Wiki
分析
Star75
18
已合并
[bugfix] 修复编译打包脚本 #72
liusa002创建于 22 天前
[bugfix] 修复编译打包脚本 #72
已合并
liusa002创建于 22 天前
从liusa002/msklfix合入到Ascend/msklmaster
L
liusa002成员
22 天前

1. 修改描述

  • 修改原因： 崇理战队对setup.py脚本提出两个issue建议，经评估可接纳。
  • 修改方案： 1、修复wheel包只包含顶层mskl包不包含子包问题；2、统一License；3、pre-commit要求用with管理open()。
  • 修改内容： 1、修复wheel包只包含顶层mskl包不包含子包问题；2、统一License；3、pre-commit要求用with管理open()。

2. 功能验证

Defaulting to user installation because normal site-packages is not writeable
Processing ./artifacts/mindstudio_kl-26.0.0-py3-none-any.whl
Installing collected packages: mindstudio-kl
Successfully installed mindstudio-kl-26.0.0

3. 代码检视

  • 要求：
    • 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。
    • committer评估是否需要在sig会议进行代码检视。
    • 参与检视的committer人员名单与检视时间。
    • 大于 1000 行代码原则上不允许合入，需进行备案。
  • 检视committer人员名单与检视时间：

4. 资料修改自检

  • 资料修改：
likedislike
Pull Request已成功合入, 合并人@ascend-robot
（感谢 liusa002 的贡献）
atomgit-bot
atomgit-bot
22 天前 评论：

变更摘要

本 PR 根据崇理战队对 setup.py 打包脚本提出的两个 issue 建议进行修复，主要聚焦于修复 wheel 包打包时仅包含顶层 mskl 包而不包含子包的问题，以及统一项目 License 声明。改动集中在 setup.py 中：引入 find_packages 动态收集包列表以替代硬编码的 packages = ['mskl']，并将 License 统一为 Mulan PSL v2，同时移除 classifiers 中原有的 MIT License 声明。

主要改动

  • 修复子包打包问题: 将 from setuptools import setup 扩展为 from setuptools import setup, find_packages，引入 find_packages 用于动态发现包。
  • 替换硬编码包列表: 将 packages = ['mskl'] 改为 packages = find_packages()，使 wheel 包不再只包含顶层 mskl 包，而是包含其全部子包。
  • 统一 License 声明: 新增 license = 'Mulan PSL v2' 配置，并在 classifiers 中删除 'License :: OSI Approved :: MIT License'，将许可证统一为木兰宽松许可证 v2。
likedislike
不准确？
atomgit-bot
atomgit-bot
22 天前 评论：

代码审查

✅ 未发现问题

likedislike
不准确？
ascend-robotascend-robot成员
22 天前 添加了label：ascend-cla/yes
ascend-robot
ascend-robot成员
22 天前 评论：

Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here

PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-Ascend/mskl ✅ wiyr0, h-glue (2/2) ✅ wiyr0 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

liusa002, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
ascend-robot
ascend-robot成员
22 天前 评论：

ascend docs pipeline is running...

likedislike
此处折叠了146条消息 查看更多
H
h-glue成员
22 天前 评论：

/lgtm

likedislike
ascend-robotascend-robot成员
22 天前 添加了label：lgtm
wiyr0
wiyr0成员
22 天前 评论：

/approve

likedislike
ascend-robotascend-robot成员
22 天前 添加了label：approved
ascend-robotascend-robot成员
22 天前 合入了pull request