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

Move all of assembly store building code to a separate class #9382

Merged
merged 5 commits into from
Oct 14, 2024

Conversation

grendello
Copy link
Contributor

@grendello grendello commented Oct 9, 2024

Simplify the task of constructing assembly stores by moving all the code
which actually stashes assemblies (and their config and debug files, if found
and required) in the assembly store to a separate utility class, AssemblyStoreBuilder.

Additionally, move a portion of assembly packaging code from BuildApk to a helper class.

This makes it easier to build the stores outside of the BuildApk task and further
simplifies BuildApk code.

This makes it possible to build assembly stores easily outside
`BuildApk`
@grendello grendello force-pushed the dev/grendel/assembly-store-builder branch from de6a7c8 to d4414ac Compare October 9, 2024 15:38
@grendello grendello merged commit 7bc14b9 into main Oct 14, 2024
58 checks passed
@grendello grendello deleted the dev/grendel/assembly-store-builder branch October 14, 2024 07:41
grendello added a commit that referenced this pull request Oct 14, 2024
Context: #9382

To further simplify `BuildApk`, move assembly compression code
to a separate utility class.  This also lets us compress assemblies
from tasks other than `BuildApk`.
@github-actions github-actions bot locked and limited conversation to collaborators Nov 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants