-
Notifications
You must be signed in to change notification settings - Fork 24
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 simple GitHub action for static checks and tests #12
Conversation
@esensar this looks ready to be merged. Is it ready to be merged? |
All checks have passed 🚀 |
I responded in https://github.com/orgs/FossifyOrg/discussions/83, but for visibility, I will also reapond here. I thought we could iron the workflow out here (as much as we can at this stage, there will probably be changes in the future) and then just copy it over to other repos when done. |
Enabled android lint, added detekt and created github workflow for PRs. Related to discussion: https://github.com/orgs/FossifyOrg/discussions/83
I think this is done for now. App is built and uploaded as actions artifact. That download link is maybe not very visible, so it may be good to add another job to post a comment on PR once APK is uploaded. Other than that, we could consider not building APKs for all changes and all PRs, but maybe we can refine that later on. |
Thank you :) |
Enabled android lint, added detekt and created github workflow for PRs.
Related to discussion: https://github.com/orgs/FossifyOrg/discussions/83