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

cleanup: refactor to use stagex #96

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

lrvick
Copy link
Contributor

@lrvick lrvick commented Jun 12, 2024

No description provided.

@lrvick lrvick requested review from james-callahan and Ulexus June 12, 2024 09:58
@lrvick lrvick force-pushed the lance/stagex-refactor branch from 18ba573 to e78b093 Compare June 12, 2024 10:04
james-callahan

This comment was marked as resolved.

@lrvick lrvick force-pushed the lance/stagex-refactor branch from e78b093 to 4249e72 Compare June 13, 2024 01:27
@lrvick
Copy link
Contributor Author

lrvick commented Jun 13, 2024

This drops the multiplatform build; shouldn't you just be able to pass the different target platforms in via GOARCH/GOOS?

We also would need to build two separate containers for these, and have multi-architecture logic. Turnkey has no use for arm64 builds and as far as I know no one external uses them either, so I am inclined to not take on that complexity in this PR, though would be happy to review a PR that adds alternative architectures back in a follow-up if there is a use for them.

@lrvick lrvick force-pushed the lance/stagex-refactor branch from 4249e72 to 37bdd0f Compare June 13, 2024 01:41
@lrvick lrvick requested a review from james-callahan June 13, 2024 01:44
@lrvick lrvick dismissed james-callahan’s stale review June 13, 2024 01:45

should be resolved for the scope of this PR

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.

2 participants