Skip to content

Add the missing mavenCentralMirror configuration - #1325

Merged
ikhoon merged 1 commit into
line:mainfrom
ikhoon:custom-maven-central2
Jul 2, 2026
Merged

Add the missing mavenCentralMirror configuration#1325
ikhoon merged 1 commit into
line:mainfrom
ikhoon:custom-maven-central2

Conversation

@ikhoon

@ikhoon ikhoon commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

repositories configuration was missing from buildscript{} in <root>/build.gradle

`repositories` configuration was missing from `buildscript{}` in
<root>/build.gradle
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f38e1517-87a0-4df6-8818-553eab7ef087

📥 Commits

Reviewing files that changed from the base of the PR and between 0c40353 and 3e082a3.

📒 Files selected for processing (1)
  • build.gradle

📝 Walkthrough

Walkthrough

Modified build.gradle to add a buildscript.repositories configuration block that reads an optional mavenCentralMirror Gradle property to determine a custom Maven repository URL, falling back to mavenCentral() when the property is absent.

Changes

Build configuration update

Layer / File(s) Summary
Buildscript repository mirror configuration
build.gradle
Reads mavenCentralMirror Gradle property via providers.gradleProperty(...).getOrNull(); configures a maven { url = ... } repository if set, otherwise falls back to mavenCentral().

Estimated code review effort: 1 (Trivial) | ~3 minutes

Related PRs: None identified.

Suggested labels: build

Suggested reviewers: None identified.

🐰

A tiny hop through build.gradle's door,
A mirror URL, nothing more,
Falling back when none is found,
Central's mirror, safe and sound.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding the missing mavenCentralMirror buildscript configuration.
Description check ✅ Passed The description is directly related to the changeset and correctly notes the missing buildscript repositories block.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jrhee17 jrhee17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 👍

@ikhoon
ikhoon merged commit e1d6e2a into line:main Jul 2, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants