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

fix: language plugins need buildEnv values #3306

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Conversation

matt2e
Copy link
Collaborator

@matt2e matt2e commented Nov 4, 2024

Without this, commands like this would not build for the right OS/Arch:
ftl deploy --build-env GOOS=linux --build-env GOARCH=amd64 --build-env CGO_ENABLED=0

@matt2e matt2e requested review from a team and alecthomas as code owners November 4, 2024 00:48
@matt2e matt2e requested review from deniseli and removed request for a team November 4, 2024 00:48
@matt2e matt2e changed the title fix: language plugins need buildEnv values passed in from build command fix: language plugins need buildEnv values Nov 4, 2024
@matt2e matt2e added the run-all A PR with this label will run the full set of CI jobs in the PR rather than in the merge queue label Nov 4, 2024
@matt2e matt2e force-pushed the matt2e/plugin-build-env branch from c16f07b to 713b38a Compare November 4, 2024 00:48
@ftl-robot ftl-robot mentioned this pull request Nov 4, 2024
@matt2e matt2e removed the run-all A PR with this label will run the full set of CI jobs in the PR rather than in the merge queue label Nov 4, 2024
@matt2e matt2e merged commit 28117b8 into main Nov 4, 2024
92 of 93 checks passed
@matt2e matt2e deleted the matt2e/plugin-build-env branch November 4, 2024 01:20
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.

1 participant