-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add Github action: android.yml - gradle ktlint and build #19
Conversation
maxme
commented
Jan 16, 2024
- Run on pull requests targetting trunk
- Run ./gradlew build
- Run ./gradlew ktlintcheck
I will wait for #18 to be merged to update that branch |
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.
❤️ Looks good! Minor comments.
- name: Grant execute permission for gradlew | ||
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.
💡 I used this configuration in another project. You can find more info here. Maybe it could be useful to avoid manually changing permissions.
- name: Grant execute permission for gradlew | |
run: chmod +x gradlew | |
- name: Setup Gradle | |
uses: gradle/gradle-build-action@v2 |
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.
TBH I don't know what's best. I took thIs from the official "Android CI" action (thus the name ;)).
It might be useful to use the wrapper (gradlew) directly to find if there is any issue with it (like a wrong config, or a incorrect version), gradle-build-action
states they cache the gradle binary, but that's not what takes the most time.
cc @gravatar/apps-infrastructure |
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.
🚀 Awesome!