Skip to content

docs: refine FEP template Packaging and Test Plan sections#52

Open
Al2O3Al2O3 wants to merge 1 commit into
flagos-ai:mainfrom
Al2O3Al2O3:fep-template-packaging-testplan
Open

docs: refine FEP template Packaging and Test Plan sections#52
Al2O3Al2O3 wants to merge 1 commit into
flagos-ai:mainfrom
Al2O3Al2O3:fep-template-packaging-testplan

Conversation

@Al2O3Al2O3

Copy link
Copy Markdown
Contributor
  • Packaging: focus on build/packaging only (not install); add supported vendors field; center on whether the feature can ship as a .whl, with build steps or a prebuilt wheel when yes, and build-from-source when no
  • Test Plan: drop image/install/setup steps; keep test commands and expected results, scoped to the new features in this release

- Packaging: focus on build/packaging only (not install); add supported
  vendors field; center on whether the feature can ship as a .whl, with
  build steps or a prebuilt wheel when yes, and build-from-source when no
- Test Plan: drop image/install/setup steps; keep test commands and
  expected results, scoped to the new features in this release

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


gcw_AUYCkC0h seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@wbavon wbavon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM — 方向正确,关注点分离后模板更清晰。

几点小建议供后续迭代参考(不阻塞合并):

  1. Test Plan 可考虑加一行可选的 Prerequisites/Environment 字段,避免验证者不知如何搭建环境
  2. Packaging 的决策树可以稍微泛化,覆盖 Docker/system package 等非-wheel 场景
  3. 注释中 python -m buildpip wheel 二选一即可,减少新手困惑

注意 CLA 状态需要解决后才能合并。

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.

3 participants