Skip to content

Remove publishing android lib#1120

Merged
gemcoder21 merged 1 commit into
mainfrom
remove-publish-android-lib
May 12, 2026
Merged

Remove publishing android lib#1120
gemcoder21 merged 1 commit into
mainfrom
remove-publish-android-lib

Conversation

@0xh3rman
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

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

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 removes the configurations for publishing the Android library to remote repositories like Maven Central and GitHub Packages, transitioning the workflow to local builds and consumption via mavenLocal(). The reviewer pointed out that the updated README is missing the specific Maven coordinates required for local integration and noted that the current build command only produces a debug variant, which may result in poor performance compared to a release build.

Comment thread README.md
Comment on lines +68 to +74
Build the Gemstone Android AAR from source and publish it to the local Maven cache:

[libraries]
gemstone = { module = "com.gemwallet.gemstone:gemstone", version.ref = "gemstone" }
```bash
just gemstone build-android
```

Add the following to your `build.gradle.kts` file:

```gradle
dependencies {
api(libs.gemstone)
}
```

#### Maven Central (Recommended)

```gradle
allprojects {
repositories {
mavenCentral()
}
}
```

#### GitHub Packages (Alternative)

Alternatively, you can use GitHub Packages (requires GitHub token):

```gradle
allprojects {
repositories {
maven {
url = uri("https://maven.pkg.github.com/gemwalletcom/core")
credentials {
username = <github_username>
password = <github_token>
}
}
}
}
```
Then consume it from `mavenLocal()` in your Android project.
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.

medium

The updated instructions for Android are missing the Maven coordinates (group, artifact, and version) required to consume the library from mavenLocal().

Additionally, the just gemstone build-android command currently only publishes the debug variant (as defined in gemstone/justfile), which uses the artifact ID gemstone-debug and version 1.0.0-debug (unless VER_NAME is set). Users should be informed of these coordinates and advised that Rust code in debug mode is significantly slower than release mode. Consider providing instructions or a command for the release variant as well (e.g., implementation("com.gemwallet.gemstone:gemstone-debug:1.0.0-debug")).

@gemcoder21 gemcoder21 merged commit 057304b into main May 12, 2026
6 checks passed
@gemcoder21 gemcoder21 deleted the remove-publish-android-lib branch May 12, 2026 02:16
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.

2 participants