-
Notifications
You must be signed in to change notification settings - Fork 555
Add SkipAndroidAPK
property to allow skipping producing an APK
#8415
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: main
Are you sure you want to change the base?
Conversation
Thanks for the PR. I can see this being useful. I have some feedback on this PR as it is though. First we tend to prefix MSBuild properties which are going to be public (i.e don't have an _ prefix) with One thing to note, there is allot of processing which happens just prior to building the apk. For example the final |
Yes, for me the C# side of the build is what I do care about for now. |
I was immediately going to suggest |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Is there an existing MSBuild target that could be called that would stop before producing the APK? I know we used to only build the If not, should there be a public target for that? It feels like this should be a target instead of a property? |
It was changed for .net 6 because the thinking was after a |
Coming back to this, there are existing targets that might be able to fulfill this use case, though they are dependent on what the goal is.
Example:
|
Sometimes, all a CI build needs is verifying that a project builds without errors. In those cases, producing an APK is a waste of time. This PR adds an option for that. Note that
BuildingInsideVisualStudio
can't be used because it can have too many other side effects.