Skip to content

[Android] Update gradle version and other changes in android apps, CI modification to auto-build Android apps and upload artifacts#11241

Merged
masahi merged 16 commits into
apache:mainfrom
argrento:gradle-update
May 25, 2022
Merged

Conversation

@argrento

@argrento argrento commented May 8, 2022

Copy link
Copy Markdown
Contributor

@argrento

argrento commented May 8, 2022

Copy link
Copy Markdown
Contributor Author

CC @TejashShah @Lyken17

@argrento argrento changed the title Update gradle version and other changes in android apps, CI modification for auto-build Android apps [Android] Update gradle version and other changes in android apps, CI modification for auto-build Android apps May 9, 2022
@TejashShah

Copy link
Copy Markdown

@Lyken17 Please take a look at it.

Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml
Comment thread .github/workflows/main.yml Outdated

@tmoreau89 tmoreau89 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @argrento for keeping the android RPC app up to date and functional. Let @KJlaccHoeUM9l if the feedback can be addressed!

Comment thread .github/workflows/main.yml Outdated
@argrento argrento changed the title [Android] Update gradle version and other changes in android apps, CI modification for auto-build Android apps [Android] Update gradle version and other changes in android apps, CI modification to auto-build Android apps and upload artifacts May 21, 2022

@tmoreau89 tmoreau89 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for addressing the comments, LGTM

@tmoreau89

Copy link
Copy Markdown
Contributor

@mehrdadh and @jwfromm wanna take a look?

@Lyken17

Lyken17 commented May 23, 2022

Copy link
Copy Markdown
Contributor

Confirmed can be built on TVM 0.8 and latest commit d146777 on Mac OSX(OpenJDK 1.8.0_312 + Clang 13) and Ubuntu(OpenJDK 1.8.0 + GCC 9).

The CI scripts LGTM, thanks for @argrento effort! Android demo and rpc are important features!

@masahi masahi merged commit 014208e into apache:main May 25, 2022
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.

6 participants