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

feat: silent parallel make build all #824

Merged
merged 17 commits into from
Jul 29, 2024
Merged

Conversation

gphorvath
Copy link

@gphorvath gphorvath commented Jul 23, 2024

Adds silent build and deploy make targets that speed up the process of standing up LeapfrogAI by parallelizing builds/deployments.

@gphorvath gphorvath requested a review from a team as a code owner July 23, 2024 22:13
Copy link

netlify bot commented Jul 23, 2024

Deploy Preview for leapfrogai-docs canceled.

Name Link
🔨 Latest commit 9a39144
🔍 Latest deploy log https://app.netlify.com/sites/leapfrogai-docs/deploys/66a7a9902fecd70008b81860

justinthelaw
justinthelaw previously approved these changes Jul 24, 2024
Copy link
Contributor

@justinthelaw justinthelaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 minor and non-mandatory comment, otherwise great idea!

Makefile Outdated Show resolved Hide resolved
@gphorvath
Copy link
Author

Much faster from zero to working deployment
image

@gphorvath gphorvath requested a review from justinthelaw July 29, 2024 01:53
Copy link
Contributor

@justinthelaw justinthelaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works beautifully! I think it is subjectively faster, although I failed to collect metrics on it...

@gphorvath gphorvath merged commit 681aafd into main Jul 29, 2024
13 checks passed
@gphorvath gphorvath deleted the feat-make-silent-parallel-build branch July 29, 2024 22:19
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