Skip to content
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

Build maui.aar on CI #8943

Merged
merged 13 commits into from
Mar 7, 2023
Merged

Build maui.aar on CI #8943

merged 13 commits into from
Mar 7, 2023

Conversation

mattleibow
Copy link
Member

@mattleibow mattleibow commented Jul 22, 2022

Description of Change

Make sure to build the maui.aar binary on CI.

The current behaviour is that the aar needs to be built locally and then checked in. This means there is no way to know what is in the aar and potential for all sorts of issues. Building this part on CI make sure the code in the repo is what is shipped.

@rmarinho
Copy link
Member

Why is this ? can we add some more documentation here?

@mattleibow
Copy link
Member Author

I see CI is failing because it needs a newer Java:

Android Gradle plugin requires Java 11 to run. You are currently using Java 1.8.

@rmarinho
Copy link
Member

rmarinho commented Aug 5, 2022

what are we missing here @mattleibow ?

@mattleibow
Copy link
Member Author

Figuring out where the android sdk is I think... The build was failing last with not being found... Or maybe it was Java 11. I wonder if we can maybe set an envvar of ANDROID_SDK_ROOT to the Android sdk path

@Redth Redth requested a review from a team as a code owner February 21, 2023 20:14
@Redth Redth changed the base branch from net6.0 to main February 21, 2023 20:14
@mattleibow mattleibow removed the request for review from a team March 7, 2023 13:58
@mattleibow mattleibow merged commit 50b509b into main Mar 7, 2023
@mattleibow mattleibow deleted the dev/android-aar branch March 7, 2023 19:33
@samhouts samhouts added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Jul 19, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2023
@samhouts samhouts added the fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants