Skip to content

Commit

Permalink
sentry: Re-enable Sentry again (on Android)
Browse files Browse the repository at this point in the history
This reverts commit e8c57e7,
except for its updates to the changelog.

Fixes: zulip#5766
  • Loading branch information
gnprice authored and chrisbobbe committed Oct 5, 2023
1 parent ed101de commit b731fc8
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
9 changes: 6 additions & 3 deletions docs/howto/release.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,12 @@ simple terminology for the process we follow with both.
tools/checkout-keystore
```

* Do *not* apply the Sentry client key from the `release-secrets`
branch; if you're already on that branch (from e.g. making the iOS
build), move off of it. See our issues #5757 and #5766.
* Apply the Sentry client key (using the local branch created for this
in initial setup):

```
git rebase @ release-secrets
```

* Build the app, as both good old-fashioned APKs and a fancy new AAB:

Expand Down
6 changes: 2 additions & 4 deletions tools/android
Original file line number Diff line number Diff line change
Expand Up @@ -64,16 +64,14 @@ do_apk() {
check_yarn_link
run_visibly yarn

# TODO(#5766): start passing -Psentry again
run_visibly tools/gradle :app:assembleRelease -Psigned
run_visibly tools/gradle :app:assembleRelease -Psigned -Psentry
}

do_aab() {
check_yarn_link
run_visibly yarn

# TODO(#5766): start passing -Psentry again
run_visibly tools/gradle :app:bundleRelease -Psigned
run_visibly tools/gradle :app:bundleRelease -Psigned -Psentry
}

case "${action}" in
Expand Down

0 comments on commit b731fc8

Please sign in to comment.