Skip to content

chore(ci): bump skywalking-eyes to fix license job Go download#1521

Open
luowanghaoyun wants to merge 1 commit into
agentscope-ai:mainfrom
luowanghaoyun:chore/ci-bump-skywalking-eyes
Open

chore(ci): bump skywalking-eyes to fix license job Go download#1521
luowanghaoyun wants to merge 1 commit into
agentscope-ai:mainfrom
luowanghaoyun:chore/ci-bump-skywalking-eyes

Conversation

@luowanghaoyun
Copy link
Copy Markdown
Contributor

Bump the pinned apache/skywalking-eyes commit so the license CI job no longer fails while downloading Go 1.18.

AgentScope-Java Version

1.1.0-SNAPSHOT

Description

The license job in maven-ci.yml uses apache/skywalking-eyes, which internally runs actions/setup-go. The previously pinned commit (e1a02359) installs Go via actions/setup-go@v2 with go-version: 1.18. When that version is missing from the toolcache manifest, setup-go falls back to the deprecated storage.googleapis.com/golang URL and fails with HTTP 403.

This PR updates both apache/skywalking-eyes and apache/skywalking-eyes/dependency to commit ab3d1fe50d23f9c82eff489297167e2dde8ba6cb on main, which includes apache/skywalking-eyes#265 (actions/setup-go@v6, Go 1.25, downloads from go.dev/dl).

How to test: Re-run the Java CI with Maven workflow; the Check License step should pass the Go setup phase.

No application code changes. Maven/Spotless checks were not run (workflow-only change).

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has been formatted with mvn spotless:apply
  • All tests are passing (mvn test)
  • Javadoc comments are complete and follow project conventions
  • Related documentation has been updated (e.g. links, examples, etc.)
  • Code is ready for review

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@luowanghaoyun luowanghaoyun requested a review from a team May 28, 2026 13:27
@codecov
Copy link
Copy Markdown

codecov Bot commented May 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@AgentScopeJavaBot AgentScopeJavaBot added bug Something isn't working area/build Build, CI, BOM, distribution labels May 29, 2026
Copy link
Copy Markdown
Collaborator

@AgentScopeJavaBot AgentScopeJavaBot left a comment

Choose a reason for hiding this comment

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

🤖 AI Review

Workflow-only chore PR bumping the pinned apache/skywalking-eyes SHA from e1a02359 to ab3d1fe5 to fix the license CI job that was failing on Go 1.18 download (deprecated storage.googleapis.com/golang returning 403). The new SHA is verified to exist on apache/skywalking-eyes's main branch and includes upstream PR #265 which migrates to actions/setup-go@v6. Both occurrences (apache/skywalking-eyes and apache/skywalking-eyes/dependency) are bumped consistently to the same full 40-char SHA. No application code, build, or runtime behavior is affected; pinning practice is preserved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build Build, CI, BOM, distribution bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants