Skip to content

build: Use Gradle credentials system#9

Closed
validcube wants to merge 3 commits intoReVanced:devfrom
validcube:dev
Closed

build: Use Gradle credentials system#9
validcube wants to merge 3 commits intoReVanced:devfrom
validcube:dev

Conversation

@validcube
Copy link
Member

@validcube validcube commented Jan 8, 2026

Switch to PasswordCredentials when during authentication with ghcr

nts:

githubPackagesUsername=
githubPackagesPassword=

List of what repository have PR submitted see: ReVanced/revanced-manager-downloaders-template#16 (comment)

Copy link

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 pull request modernizes the Gradle authentication mechanism by switching from manual environment variable handling to Gradle's built-in PasswordCredentials system for GitHub Packages authentication.

Key changes:

  • Simplified build.gradle.kts by replacing explicit credential configuration with credentials(PasswordCredentials::class)
  • Updated GitHub Actions workflows to use Gradle project properties (ORG_GRADLE_PROJECT_* prefix) instead of direct environment variables
  • Applied changes consistently across both release and pull request build workflows

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 4 comments.

File Description
build.gradle.kts Switched from explicit username/password credential configuration to Gradle's PasswordCredentials class
.github/workflows/release.yml Updated environment variables to use Gradle project property convention for credentials
.github/workflows/build_pull_request.yml Updated environment variables to use Gradle project property convention for credentials

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

@oSumAtrIX
Copy link
Member

Implemented in #11

@oSumAtrIX oSumAtrIX closed this Feb 7, 2026
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.

3 participants