-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Android CI #39
base: master
Are you sure you want to change the base?
Android CI #39
Conversation
I have added github actions code for running Android CI with placeholder files for
|
You may have to create google-services.json.sample and keystore-properties.sample and add
In build steps to avoid conflicting with actual config files |
Updated with sample files to avoid conflicts |
Squash your commits please. |
./gradlew assembleDebug Create keystore.properties Create google-services.json Update google-services.json Rename google-services.json to app/google-services.json Update android.yml Update google-services.json Update android.yml Update google-services.json Update Contributors.md if it merges, my name would automatically get added to contributors.md Rename keystore.properties to keystore.properties.sample Rename app/google-services.json to google-services.json.sample Update android.yml Update android.yml
Squashed commits into 1 |
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 great
Unit tests are written for some modules, worth adding that gradle Task to CI ? |
add test
Added test command to ci |
Co-authored-by: HARSHIT GUPTA <harshitsaamu@gmail.com>
Who can merge this? Does anybody know? |
@ahujamoh Only owner and collaborators can merge. So only one of them if I'm right: @niharika2810 |
@delhiamitk can you please merge this? |
I believe they will not merge any PRs or solve any issues over here as this is like a piece of reference for encouraging the bug bounty program and making the source code open.
The project is not being maintained using GitHub over here.
…On Sun, May 31, 2020 at 6:26 PM Mohit Ahuja ***@***.***> wrote:
@delhiamitk <https://github.com/delhiamitk> can you please merge this?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADFM5VQOCDJWI4UAYFZIPU3RUJHYPANCNFSM4NK74L7A>
.
--
Thanks and Regards,
Subhrodip Mohanta. <https://subho.xyz>
|
No description provided.