Skip to content

.1030693119606345:82670e7adc353364f8582e69fb008425_69dda4f06a20c328a5c2f58a.69deda1f0d6309e3b0618d0d.69deda1f239c216203e108a4:Trae CN.T(2026/4/15 08:21:51)#122

Closed
cytxnyu wants to merge 1 commit into
smarlhens:mainfrom
cytxnyu:trae-1
Closed

.1030693119606345:82670e7adc353364f8582e69fb008425_69dda4f06a20c328a5c2f58a.69deda1f0d6309e3b0618d0d.69deda1f239c216203e108a4:Trae CN.T(2026/4/15 08:21:51)#122
cytxnyu wants to merge 1 commit into
smarlhens:mainfrom
cytxnyu:trae-1

Conversation

@cytxnyu
Copy link
Copy Markdown
Contributor

@cytxnyu cytxnyu commented Apr 15, 2026

  • 合并pr.yml到ci.yml,简化工作流配置
  • 添加lint和测试步骤到CI流程
  • 更新README.md添加验证步骤说明
  • 移除pre-commit的ci.skip配置
  • 锁定uv版本为0.10.3

- 合并pr.yml到ci.yml,简化工作流配置
- 添加lint和测试步骤到CI流程
- 更新README.md添加验证步骤说明
- 移除pre-commit的ci.skip配置
- 锁定uv版本为0.10.3
@smarlhens
Copy link
Copy Markdown
Owner

Hi @cytxnyu, thanks for the contribution.

I've integrated your work across several commits in #123 (now merged on main):

  • docs: add validation section to readme (7907378) — adapted your README validation section into command-only subsections, co-authored to you.
  • ci: merge pr workflow into ci workflow (478353f) — applied your consolidation of pr.yml into ci.yml with push: main + pull_request: **, co-authored to you.
  • ci: add lint-and-test job & gate build on it (acb97db) — adopted your lint-and-test job structure with build and build-image depending on it, co-authored to you.

Two parts I didn't carry over:

  • The required-version = "==0.10.3" pin — uv was bumped to 0.11.8 in a separate commit, with the floor set to >=0.11.5 (minimum supporting Python 3.14.4).
  • Removing ci.skip: [pytest] from .pre-commit-config.yaml — kept it so pre-commit.ci stays fast; pytest now runs in the GitHub Actions lint-and-test job instead.

Closing this PR since the work is upstreamed. Thanks again!

@smarlhens smarlhens closed this Apr 29, 2026
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