Skip to content

feat(jsii-pacmak): allow disabling go build conditionally#4196

Merged
mergify[bot] merged 3 commits intoaws:mainfrom
DanielMSchmidt:skip-go-build-conditionally
Jul 26, 2023
Merged

feat(jsii-pacmak): allow disabling go build conditionally#4196
mergify[bot] merged 3 commits intoaws:mainfrom
DanielMSchmidt:skip-go-build-conditionally

Conversation

@DanielMSchmidt
Copy link
Contributor

@DanielMSchmidt DanielMSchmidt commented Jul 26, 2023

In CDKTF this step causes our examples to take up to 24GB of Ram and ~5 minutes to have the bindings generated. We split our code into a lot of modules, so if a user just ran go run ... and only the parts needed would be compiled they can save a lot of time and resources.

Co-authored-by: Romain Marcadier <romain.muller@telecomnancy.net>
@RomainMuller
Copy link
Contributor

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jul 26, 2023

update

✅ Branch has been successfully updated

@mergify
Copy link
Contributor

mergify bot commented Jul 26, 2023

Thank you for contributing! ❤️ I will now look into making sure the PR is up-to-date, then proceed to try and merge it!

@mergify mergify bot added the pr/ready-to-merge This PR is ready to be merged. label Jul 26, 2023
@mergify
Copy link
Contributor

mergify bot commented Jul 26, 2023

Merging (with squash)...

@mergify mergify bot merged commit 7fc7987 into aws:main Jul 26, 2023
@mergify mergify bot removed the pr/ready-to-merge This PR is ready to be merged. label Jul 26, 2023
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