Skip to content

Add CI workflows for build, PR checks, and pre-release#17

Open
XChen446 wants to merge 4 commits into
TheCommandCraft:mainfrom
XChen446:main
Open

Add CI workflows for build, PR checks, and pre-release#17
XChen446 wants to merge 4 commits into
TheCommandCraft:mainfrom
XChen446:main

Conversation

@XChen446

Copy link
Copy Markdown
Contributor
  • 配置构建工作流,支持推送和标签触发,使用JDK 21构建并上传JAR制品
  • 设置PR检查工作流,包含依赖缓存隔离和测试执行
  • 创建预发布工作流,在构建成功后自动创建GitHub预发布版本
  • 使用Maven缓存优化构建性能,配置30天制品保留策略
  • 实现PR依赖缓存严格隔离,防止缓存污染和安全问题
  • 添加测试报告上传功能,便于质量监控和问题排查

- 配置构建工作流,支持推送和标签触发,使用JDK 21构建并上传JAR制品
- 设置PR检查工作流,包含依赖缓存隔离和测试执行
- 创建预发布工作流,在构建成功后自动创建GitHub预发布版本
- 使用Maven缓存优化构建性能,配置30天制品保留策略
- 实现PR依赖缓存严格隔离,防止缓存污染和安全问题
- 添加测试报告上传功能,便于质量监控和问题排查
@TheCommandCraft

Copy link
Copy Markdown
Owner

Thanks for setting up CI! A few things need addressing before this can merge:

  1. build.yml skips tests on every push including main. Tests should run on main, either change to mvn clean verify or add a separate job that runs tests on main pushes.

  2. The test report upload path is target/surefire-reports/ but this is a multi-module project, the actual reports are under vanishpp-paper/target/surefire-reports/. Change to **/target/surefire-reports/.

  3. Please translate the inline Chinese comments to English to stay consistent with the rest of the codebase.

  4. Minor: all three files are missing a trailing newline.

@XChen446

Copy link
Copy Markdown
Contributor Author

Thks for pointing out ,review it again plz

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