Skip to content

Conversation

@kalverra
Copy link
Collaborator

@kalverra kalverra commented Dec 5, 2025

Passes in a github token for auth when installing Sui from GitHub

Copilot AI review requested due to automatic review settings December 5, 2025 15:13
@kalverra kalverra requested review from a team as code owners December 5, 2025 15:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds GitHub token authentication when installing the Sui CLI from GitHub, addressing DX-2442. The change ensures authenticated API requests to avoid rate limiting during CI runs.

Key Changes:

  • Added github-token input parameter to the setup-sui action
  • Set GitHub token environment variables during Sui installation
  • Passed GITHUB_TOKEN secret from the CI workflow to the action

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/ci-core.yml Passes GitHub token secret to setup-sui action
.github/actions/setup-sui/action.yaml Adds token input parameter and sets environment variables for authenticated installation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Tofel
Tofel previously approved these changes Dec 5, 2025
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@kalverra kalverra added this pull request to the merge queue Dec 5, 2025
Merged via the queue into develop with commit 5809af1 Dec 5, 2025
96 of 97 checks passed
@kalverra kalverra deleted the suiAuth branch December 5, 2025 16:10
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.

4 participants