-
Notifications
You must be signed in to change notification settings - Fork 409
Adding Android Developer Verification Sample #110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…dates throughout. Update readme.md files with deprecation warnings.
Summary of ChangesHello @TKBurner, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a new, essential Android Developer Verification Onboarding sample, providing a clear template for developers to correctly set up their projects for the Android developer verification proof of ownership process. Simultaneously, it addresses the maintenance status of older security samples by officially deprecating them and updating relevant documentation to reflect their end-of-life, ensuring users are aware of which samples are actively supported. Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request introduces a new sample project for Android Developer Verification, which is a great addition. It also includes updates to deprecate older samples and adds license headers across various files. My review focuses on the new sample project and I've found a few issues: a critical error in an instrumented test that will cause it to fail, incorrect metadata in a packaging file, and a missed opportunity to follow best practices in the build configuration. The changes to deprecate old samples and update licenses look good.
.../src/androidTest/java/com/androiddeveloperverification/onboarding/ExampleInstrumentedTest.kt
Outdated
Show resolved
Hide resolved
| apiRefs: | ||
| - android:android.hardware.fingerprint.FingerprintManager | ||
| - android:android.hardware.fingerprint.FingerprintManager.AuthenticationCallback | ||
| - android:android.hardware.fingerprint.FingerprintManager.CryptoObject | ||
| - android:android.security.KeyGenParameterSpec | ||
| - android:java.security.KeyStore | ||
| - android:javax.crypto.Cipher | ||
| - android:javax.crypto.KeyGenerator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| buildTypes { | ||
| release { | ||
| isMinifyEnabled = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…a/com/androiddeveloperverification/onboarding/ExampleInstrumentedTest.kt Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
.../src/androidTest/java/com/androiddeveloperverification/onboarding/ExampleInstrumentedTest.kt
Outdated
Show resolved
Hide resolved
...nOnboarding/app/src/test/java/com/androiddeveloperverification/onboarding/ExampleUnitTest.kt
Outdated
Show resolved
Hide resolved
|
|
||
| - name: Make gradlew executable | ||
| run: chmod +x ./gradlew | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not now, but in a future PR, please consider configuring Spotless and then adding a required build step. This will help enforce code style and copyright headers. You can add the enforcement to this file later:
- name: Check spotless
run: ./gradlew spotlessCheck
cartland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Please consider adding Spotless in a future update, but no need to do it now.
Add Android Developer Verification onboarding sample. Add licenses updates throughout. Update readme.md files with deprecation warnings.