Skip to content

Use per-project Sentry configuration (#4818)#4912

Merged
geekygecko merged 1 commit intorelease/8.4from
mehow/task/sentry-sync
Jan 27, 2026
Merged

Use per-project Sentry configuration (#4818)#4912
geekygecko merged 1 commit intorelease/8.4from
mehow/task/sentry-sync

Conversation

@MiSikora
Copy link
Contributor

@MiSikora MiSikora commented Jan 23, 2026

This syncs Sentry configuration changes with the current release branch. See #4818 for rationale.

@MiSikora MiSikora added this to the 8.4 ❄️ milestone Jan 23, 2026
@MiSikora MiSikora requested a review from a team as a code owner January 23, 2026 15:49
Copilot AI review requested due to automatic review settings January 23, 2026 15:49
@MiSikora MiSikora requested review from geekygecko and removed request for a team January 23, 2026 15:49
@dangermattic
Copy link
Collaborator

dangermattic commented Jan 23, 2026

1 Warning
⚠️ This PR is assigned to the milestone 8.4 ❄️. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

@MiSikora MiSikora added [Type] Tooling Related to the Gradle build scripts and the setup or maintenance of the project build process. [Area] Tooling and removed releases labels Jan 23, 2026
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

Updates the build to use per-project Sentry configuration (auth/org + per-module project name) instead of relying on a shared sentry.properties, aligned with the current release branch.

Changes:

  • Adds module-specific sentry.projectName configuration for app, wear, and automotive.
  • Adds shared Sentry auth/org configuration in the root Gradle Sentry setup.
  • Moves Sentry secrets handling into secret.properties (and updates .configure inputs accordingly).

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
app/build.gradle.kts Sets Sentry projectName from a Gradle property; updates release warning logic.
wear/build.gradle.kts Sets Sentry projectName from a Gradle property; updates release warning logic.
automotive/build.gradle.kts Sets Sentry projectName from a Gradle property; adds release warning logic.
build.gradle.kts Renames and extends common Sentry configuration to include auth token + org.
dependencies.gradle.kts Exposes Sentry secrets/projects from secret.properties as Gradle properties.
.configure Stops copying/encrypting sentry.properties; updates pinned hash.
.configure-files/secret.properties.enc Adds updated encrypted secrets payload containing the new Sentry-related properties.
Comments suppressed due to low confidence (1)

.configure:15

  • This removes the sentry.properties secret from .configure, but the repo still contains .configure-files/sentry.properties.enc. To avoid confusion and keeping an unused secret artifact around, consider deleting the encrypted file as part of this change (or documenting why it must remain).
  "pinned_hash": "f8db88d1a1d1149ce3a144b7c629e36717c885ea",
  "files_to_copy": [
    {
      "file": "android/pocket-casts/secret.properties",
      "destination": "secret.properties",
      "encrypt": true
    },
    {
      "file": "android/pocket-casts/google-services.json",
      "destination": "app/google-services.json",
      "encrypt": true
    },

@geekygecko geekygecko merged commit c609746 into release/8.4 Jan 27, 2026
32 of 34 checks passed
@geekygecko geekygecko deleted the mehow/task/sentry-sync branch January 27, 2026 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Area] Tooling [Type] Tooling Related to the Gradle build scripts and the setup or maintenance of the project build process.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants