Skip to content
This repository was archived by the owner on Jun 16, 2026. It is now read-only.

chore: fix code/doc inaccuracies across project#13

Merged
TommyLike merged 2 commits into
mainfrom
feat/support-release
Jun 8, 2026
Merged

chore: fix code/doc inaccuracies across project#13
TommyLike merged 2 commits into
mainfrom
feat/support-release

Conversation

@TommyLike

@TommyLike TommyLike commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

Project-wide audit + fix of inaccurate code, comments, and stale references.

Changes

Fix File
InjectAuth comment: add missing Jenkins + EUR auth internal/auth/resolver.go
go.mod + all imports: cncf/coraopensourceways/cora 22 files
.gitignore: add dist/ .gitignore
CLAUDE.md: add GitHub/Jenkins/EUR to auth docs claude-specification/project/CLAUDE.md
Remove stale spec/spec-optimization-design.md deleted
Fix outdated Etherpad view pads/listpad/list-all-pads internal/view/builtin.go

Files changed (23)

 .gitignore, CLAUDE.md, cmd/cora/main.go, cmd/cora/services_cmd.go,
 cmd/smoke/main.go, go.mod, go.sum, internal/auth/*, internal/builder/*,
 internal/config/*, internal/executor/*, internal/output/*,
 internal/registry/*, internal/spec/*, internal/view/builtin.go,
 pkg/errs/*, spec/spec-optimization-design.md (deleted)

Quality

  • go build ./...
  • go test ./... (all pass)
  • go vet ./... (clean)
  • gofmt -l . (clean)

Generated with Claude Code

@opensourceways-bot

Copy link
Copy Markdown

Welcome To opensourceways Community

Hey @TommyLike , thanks for your contribution to the community.

Bot Usage Manual

I'm the Bot here serving you. You can find the instructions on how to interact with me at Here . That means you can comment below every pull request or issue to trigger Bot Commands.

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

TommyLike, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

Linking Issue Notice

@TommyLike , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request migrates the Go module path from github.com/cncf/cora to github.com/opensourceways/cora across all source files and the go.mod file. It also deletes the spec-optimization-design.md document, updates CLAUDE.md to reflect this removal and document new authentication header injections (GitHub, Jenkins, EUR), updates dependency versions in go.sum, and modifies a view key for Etherpad. The review feedback correctly identifies a redundant duplicate entry for the 'dist/' directory in the .gitignore file.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread .gitignore
smoke-report/
.playwright-mcp/
.claude/worktrees/
dist/

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The 'dist/' directory is already ignored on line 14. This duplicate entry is redundant and can be safely removed.

TommyLike and others added 2 commits June 8, 2026 08:38
- InjectAuth comment: add missing Jenkins + EUR auth descriptions
- go.mod: cncf/cora → opensourceways/cora (matching actual repo)
- All Go imports: cncf/cora → opensourceways/cora
- .gitignore: add dist/
- CLAUDE.md: add GitHub/Jenkins/EUR to auth docs, remove stale
  spec-optimization-design.md reference
- Remove spec/spec-optimization-design.md (outdated design doc)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@TommyLike TommyLike force-pushed the feat/support-release branch from 9b613c4 to c4b84c0 Compare June 8, 2026 00:44
@TommyLike TommyLike changed the title feat: cross-platform release + code/doc cleanup chore: fix code/doc inaccuracies across project Jun 8, 2026
@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

TommyLike, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@TommyLike TommyLike merged commit f669927 into main Jun 8, 2026
7 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants