CI: 新增原生环境打包 & 启动日志改进#190
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR expands the release pipeline to build Docker and native runtime packages, and improves startup output by logging the frontend access URL.
Changes:
- Reworks Maven CI into shared build, Docker image, and Linux/Windows native package jobs.
- Adds startup logging for the page access address by reading proxy configuration.
- Ignores
.spec-workflow/local tooling files.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
.github/workflows/maven.yml |
Adds artifact sharing, multi-platform Docker build, jlink-based native packaging, and release uploads. |
web-service/src/main/java/cn/qaiu/lz/AppMain.java |
Reads proxy config to derive frontend URL and logs it after playground parser loading. |
.gitignore |
Adds .spec-workflow/ to ignored paths. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+92
to
+95
| Path configPath = Path.of("resources", configFile); | ||
| if (!Files.exists(configPath)) { | ||
| configPath = Path.of(configFile); | ||
| } |
Comment on lines
189
to
192
| log.info("服务已启动，可通过 {} 访问页面", accessAddr); | ||
| }).onFailure(e -> { | ||
| log.error("加载演练场解析器列表失败", e); | ||
| }); |
| if: runner.os == 'Windows' | ||
| shell: pwsh | ||
| run: | | ||
| Compress-Archive -Path native-package/netdisk-fast-download/* -DestinationPath "${{ matrix.artifact-name }}.zip" |
| if (addr == null || addr.isBlank()) { | ||
| addr = "http://127.0.0.1:" + jsonObject.getJsonObject(ConfigConstant.SERVER).getInteger("port", 6400); | ||
| } | ||
| // 读取代理配置获取前端页面端口（同步读文件，避免阻塞 event loop） |
- 配置文件查找顺序与 Deploy 保持一致（先当前目录，再 resources/） - 页面地址日志改用 onComplete，无论演练场加载成功失败均输出 - Windows ZIP 移除 /* 通配符，与 Linux 保持一致的顶层目录结构 - 修正注释：同步读文件会阻塞 event loop，不再声称'避免阻塞' - YAML 正则和 jdeps 回退列表补充适用范围说明
Contributor
Author
Copilot Review 修复Copilot 提出的 4 条 review 已全部处理，修复已提交到本 PR 分支（0feb8e7）：
额外补充：YAML 正则和 jdeps 回退模块列表均增加了适用范围注释。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
变更摘要
CI: 新增 Linux/Windows 原生环境打包 (jlink + 精简 JRE)
AppMain: 启动日志增加前端页面访问地址提示