feat(android-nav3): [Android Nav3 1] Create the navigation3 module - #6129
0xadam-brown merged 3 commits into
Conversation
|
📲 Install BuildsAndroid
|
Establish the new Android Navigation 3 integration module and register it with the build and publication metadata. This keeps the first step focused on the module boundary itself, without introducing generated API surface or implementation code yet. Co-Authored-By: OpenCode <noreply@opencode>
e5d13ee to
1b98b50
Compare
AGP 9 only generates unit tests for the configured test build type. Because this module disables the debug variant, point unit tests at release so future tests run instead of being skipped.
runningcode
left a comment
There was a problem hiding this comment.
LGTM! just some nits
markushi
left a comment
There was a problem hiding this comment.
Looking good! Not sure (yet) if it's covered by a follow up PR, but we probably need to add the module to the following too:
.craft.ymlAGENTS.md.aarconfigsentry-java/buildSrc/src/main/java/Config.kt
Lines 95 to 102 in 113f678
- bug report template https://github.com/getsentry/sentry-java/blob/main/.github/ISSUE_TEMPLATE/bug_report_android.yml
Good callouts! I'm leaving Updated AGENTS.md and the bug report template (and already added to .aar config) 👍 |
7300862
into
feat/sentry-nav3-effect-stack
PR Stack (Android Nav3)
📜 Description
Create the new
sentry-android-navigation3module and wire it into the build.This keeps the first step of the stack focused on the module boundary itself: Gradle registration,
module layout, and Android library scaffolding, without introducing implementation code yet.
💡 Motivation and Context
Before we can add any Nav3 behavior, we need a clean module home for it. Splitting that setup out
first makes the rest of the stack easier to review and reason about.
addresses: JAVA-274
💚 How did you test it?
Built it + verified the config works correctly with the remaining PRs in this stack.
📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps
Layer route modeling on top of the module scaffold.
#skip-changelog