Skip to content

ci: verify go releases end to end - #10

Merged
kantorcodes merged 1 commit into
mainfrom
feat/go-release-automation
Apr 4, 2026
Merged

kantorcodes merged 1 commit into
mainfrom
feat/go-release-automation

Conversation

@kantorcodes

Copy link
Copy Markdown
Member

Summary

  • ensure GitHub releases are reconciled even on reruns or existing tags
  • append the canonical HOL docs URL to generated release notes
  • verify module publishing by waiting for proxy.golang.org and pkg.go.dev indexing

Verification

  • parsed .github/workflows/ci.yml with PyYAML
  • git diff --check

Signed-off-by: Michael Kantor <6068672+kantorcodes@users.noreply.github.com>
@kantorcodes
kantorcodes requested a review from HGraphPunks as a code owner April 4, 2026 19:10
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

Comment thread .github/workflows/ci.yml

tag="${{ steps.release_tag.outputs.tag }}"
docs_line="- Standards SDK docs: ${HOL_SDK_DOCS_URL}"
release_notes="$(printf '## Documentation\n%s\n' "${docs_line}")"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WARNING: --generate-notes and --notes flags used together when creating release. When both are specified, --notes takes precedence and GitHub will not generate any release notes.

The documentation header with --generate-notes will be lost.

Comment thread .github/workflows/ci.yml
fi

body="$(gh release view "${tag}" --json body --jq '.body // ""')"
if printf '%s' "${body}" | grep -Fq "${HOL_SDK_DOCS_URL}"; then

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SUGGESTION: The grep -Fq check is exact string matching. Consider using a case-insensitive search (grep -Fi) to handle potential case variations in the release body format.

@kilo-code-bot

kilo-code-bot Bot commented Apr 4, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: 2 Issues Found | Recommendation: Address comments before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 1
Issue Details (click to expand)

WARNING

File Line Issue
.github/workflows/ci.yml 221 --generate-notes and --notes flags conflict when creating release. The --notes flag takes precedence, so GitHub-generated release notes won't be included.

SUGGESTION

File Line Issue
.github/workflows/ci.yml 233 Consider using case-insensitive grep (-Fi) instead of -Fq for more robust URL matching in release body.
Other Observations (not in diff)

No other issues found in the codebase that couldn't receive inline comments.

Files Reviewed (1 file)
  • .github/workflows/ci.yml - 2 issues

Fix these issues in Kilo Cloud


Reviewed by minimax-m2.5-20260211 · 239,437 tokens

@kantorcodes
kantorcodes merged commit 1c3608f into main Apr 4, 2026
10 checks passed
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.

1 participant