ci(deps): bump codecov/codecov-action from 3 to 5#1
Closed
dependabot[bot] wants to merge 2 commits into
Closed
Conversation
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 3 to 5. - [Release notes](https://github.com/codecov/codecov-action/releases) - [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md) - [Commits](codecov/codecov-action@v3...v5) --- updated-dependencies: - dependency-name: codecov/codecov-action dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Author
AssigneesThe following users could not be added as assignees: Please fix the above issues or remove invalid values from |
dependabot
Bot
added
dependencies
Author
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
dependabot
Bot
deleted the
dependabot/github_actions/codecov/codecov-action-5
branch
zhaojing1987
added a commit
that referenced
this pull request
Jan 12, 2026
All findings addressed: - HIGH #1: Server IPv4 access - Added troubleshooting guide (Issue #5) - HIGH #2: Temporal UI - Added port conflict troubleshooting (Issue #6) - HIGH #3: Agent healthcheck - Documented metrics endpoint issue (Issue #7) - MEDIUM #4: README validation - Added network troubleshooting - MEDIUM #5: .env.example - Expanded from 27 to 88 lines with detailed comments - MEDIUM #6: Completion Notes - Updated with honest AC assessment - MEDIUM #7: README testing - Added prerequisite clarifications - LOW #8: Agent healthcheck - Optimized wget command (simplified) - LOW #9: Story doc - Updated AC2 CORS example - LOW #10: Completion Notes - Clarified /v1/nodes endpoint status AC Achievement: - AC3, AC4, AC6, AC7: 100% achieved - AC1 (full stack): Partial - Agent unhealthy (metrics not implemented) - AC2 (Temporal UI): Partial - Container runs but port may conflict - AC5 (endpoints): Partial - Server listens IPv6, host access may fail Known Issues Documented: 1. Server listens on IPv6 (:::8080) not IPv4 (0.0.0.0:8080) 2. Agent metrics HTTP server not implemented (9090/metrics) 3. Temporal UI port mapping may conflict Files Modified: - deployments/.env.example: Expanded with detailed comments (88 lines) - deployments/README.md: Added 3 troubleshooting scenarios - deployments/docker-compose.yaml: Optimized Agent healthcheck - docs/sprint-artifacts/8-2: Updated Completion Notes Story Status: review -> done
zhaojing1987
added a commit
that referenced
this pull request
Jan 12, 2026
修复Story 8-3配置管理的8个审查问题: P1问题修复 (文档和示例): - #1: 添加TOML格式文档示例和格式对比表格 - #2: 创建examples/configs/config-minimal.toml示例文件 - #6: 补充Agent环境变量文档,添加Docker/K8s部署示例 P2/P3问题修复 (文档优化): - #3: 更新Completion Notes,包含详细AC达成表、技术细节、测试覆盖 - #4: 添加配置文件路径章节,说明Server/Agent默认路径差异原因 - #7: 修正Agent ID注释说明(移除未实现的auto-generated承诺) - #8: 配置示例已有详细注释,无需额外修改 文档增强: - 在docs/configuration.md添加格式对比表格(YAML vs TOML) - 补充Agent环境变量使用示例(bash/Docker/Kubernetes) - 添加配置文件路径说明章节(默认路径/指定路径/不存在行为) - Completion Notes扩展至200+行,包含AC达成度、技术细节、已知限制 文件修改: - docs/configuration.md: +153行 (TOML示例、Agent环境变量、路径说明) - docs/sprint-artifacts/8-3-configuration-management.md: +200行 (Completion Notes) - examples/configs/config-minimal.toml: 新增39行TOML格式示例 - pkg/config/config.go: 修正AgentConfig.ID注释说明 审查结果: 8个问题全部修复,代码质量8.5/10 Related: Story 8-3
zhaojing1987
added a commit
that referenced
this pull request
Jan 28, 2026
…gger field BREAKING CHANGE: Remove 'on' field from Workflow YAML schema to implement API-driven architecture where trigger configuration is managed via REST API. Changes: - Remove Workflow.On field and trigger types (TriggerConfig, PushTrigger, etc) - Update JSON Schema: remove 'on' from required fields and property definitions - Update workflow-schema.json description to reflect API-driven architecture - Fix renderer.go, parser_test.go, renderer_test.go to remove On field usage - Remove 'on:' field from 32 test/example YAML files across testdata/ and examples/ - Add architecture refactoring section to Story 1-3 documentation Benefits: - Separation of concerns: YAML defines workflow logic, API manages triggers - Flexibility: Same workflow supports multiple trigger configurations - Maintainability: Trigger changes don't affect workflow definitions - Extensibility: Support new trigger types without YAML schema changes Migration: - Users must remove 'on:' field from existing YAML files - Configure triggers via Schedule API (Story 1.10) or Webhook API (Story 1.11) Related Stories: #1.10 (Schedule API), #1.11 (Webhook API) Files changed: 48 (6 core code + 42 tests/examples) All tests passing: pkg/dsl (0.813s), internal/api (0.032s)
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
1 participant
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.
Bumps codecov/codecov-action from 3 to 5.
Release notes
Sourced from codecov/codecov-action's releases.
... (truncated)
Changelog
Sourced from codecov/codecov-action's changelog.
... (truncated)
Commits
5a10915chore(release): 5.5.1 (#1873)3e0ce21fix: overwrite pr number on fork (#1871)c4741c8build(deps): bump actions/checkout from 4.2.2 to 5.0.0 (#1868)17370e8build(deps): bump github/codeql-action from 3.29.9 to 3.29.11 (#1867)18fdacffix: update to use local app/ dir (#1872)206148cdocs: fix typo in README (#1866)3cb13a1Document acodecov-cliversion reference example (#1774)a4803c1build(deps): bump github/codeql-action from 3.28.18 to 3.29.9 (#1861)3139621build(deps): bump ossf/scorecard-action from 2.4.1 to 2.4.2 (#1833)fdcc847chore(release): 5.5.0 (#1865)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)