wordpress-mobile / WordPress-Android

WordPress for Android
http://android.wordpress.org
GNU General Public License v2.0
2.99k stars 1.33k forks source link

Don't send sentry mapping files from local machines #21452

Closed wzieba closed 1 week ago

wzieba commented 1 week ago

Description

When building release on local machine, we shouldn't send debug files to Sentry as it's not needed: clutters the Sentry dashboard and slows down the local build. This tiny PR addresses this problem.

Testing

  1. ./gradlew assembleJetpackVanillaRelease does not send debug files (see console output)
  2. CI=true ./gradlew assembleJetpackVanillaRelease sends debug files
wpmobilebot commented 1 week ago
Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr21452-79fe4cb
Commit79fe4cbc8a02080c9c1b1096a63237a2eec1da1c
Direct Downloadjetpack-prototype-build-pr21452-79fe4cb.apk
Note: Google Login is not supported on these builds.
wpmobilebot commented 1 week ago
WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr21452-79fe4cb
Commit79fe4cbc8a02080c9c1b1096a63237a2eec1da1c
Direct Downloadwordpress-prototype-build-pr21452-79fe4cb.apk
Note: Google Login is not supported on these builds.
sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud