Skip to content

Conversation

riggaroo
Copy link
Collaborator

@riggaroo riggaroo commented Apr 25, 2024

Switching to local debug.keystore to prevent issues with installing debug app on device.

@riggaroo riggaroo marked this pull request as ready for review April 25, 2024 15:24
@riggaroo riggaroo requested a review from a team as a code owner April 25, 2024 15:24
@riggaroo riggaroo requested a review from dturner April 25, 2024 15:24
@@ -34,9 +34,9 @@ android {
}

signingConfigs {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we just remove this block altogether? I think Android Studio already defaults to the local key with these parameters for debug builds.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question is if the benchmark buildType allows it, but I assume there's default signingConfig even if it's not defined.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark requires release build, and release requires a certificate :-/ Not sure how else to work around this

@riggaroo riggaroo merged commit ac8340d into main Apr 29, 2024
@riggaroo riggaroo deleted the bugfix/fix-debug-keystores branch April 29, 2024 11:09
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.

4 participants