Skip to content

Added the hockey app update activity for debug builds#20

Merged
Plinzen merged 1 commit intomasterfrom
feature/hockey_app_activity
Sep 8, 2015
Merged

Added the hockey app update activity for debug builds#20
Plinzen merged 1 commit intomasterfrom
feature/hockey_app_activity

Conversation

@jleidgens
Copy link
Contributor

Fixing #19

@Plinzen
Copy link
Contributor

Plinzen commented Sep 8, 2015

Hi,
passt soweit ganz gut. Das einzige was man noch "verbessern" könnte. Im CrashTracker wird die Registierung auf Updates ja immer dann gemacht, wenn es sich um einen Debug-Build (und nicht IDE Build handelt). Wenn man es jetzt genau nimmt, dann hat das Attribut BuildConfig.DEBUG mit dem BuildType debug erstmal nichts zu tun und wenn ich einen neuen BuildType anlege und mir nicht bewusst ist, dass ich in meinem Manifest noch die Hockey-UpdateActivity registrieren muss, dann crasht evtl. es beim Bauen. Fieserweise entdeckt man das dann aber evtl. erst dann, wenn am Jenkins (nicht in der IDE) gebaut wird.

Das ist jetzt zwar nur eine Feinheit, aber wir sollten in unserer Baseapp versuchen, Fallstricke zu vermeiden, oder? Was meinst du dazu? Ich kann den PR auch gerne mergen, aber ich wollte dich vorher mal darauf hinweisen.

VG Christopher

@jleidgens
Copy link
Contributor Author

Good point. Can you merge the pull request and create a new issue for that one?

Plinzen added a commit that referenced this pull request Sep 8, 2015
Added the hockey app update activity for debug builds
@Plinzen Plinzen merged commit 25676f9 into master Sep 8, 2015
@Plinzen Plinzen deleted the feature/hockey_app_activity branch September 8, 2015 10:36
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

Comments