这是indexloc提供的服务，不要输入任何密码
Skip to content

Large Monorepo Benchmark #740

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Feb 15, 2022
Merged

Large Monorepo Benchmark #740

merged 3 commits into from
Feb 15, 2022

Conversation

gsoltis
Copy link
Contributor

@gsoltis gsoltis commented Feb 14, 2022 •
edited
Loading

Runs a set of benchmarks against a large monorepo example.

Benchmarks include:

  • A clean, zero-state build* of the monorepo
  • Building the unchanged monorepo with a full cache
  • Building the monorepo with a source code edit on top of a previously-built cache
  • Building the monorepo with a dependency graph edit on top of a previously-build cache

*: Note that we only do this once, and with concurrency set to 1. A full concurrent build currently OOMs

@vercel
Copy link

vercel bot commented Feb 14, 2022 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/vercel/turbo-site/JCsX7ZL71R3DWJKqyJjgrsck1J75
✅ Preview: https://turbo-site-git-gsoltis-largemonorepo.vercel.sh

@vercel
Copy link

vercel bot commented Feb 14, 2022

Deployment failed with the following error:

Resource is limited - try again after in 2 minutes (more than 150, code: "api-deployments-paid-per-hour").

@vercel
Copy link

vercel bot commented Feb 14, 2022

Deployment failed with the following error:

Resource is limited - try again after in 1 minute (more than 150, code: "api-deployments-paid-per-hour").

@vercel
Copy link

vercel bot commented Feb 14, 2022

Deployment failed with the following error:

Resource is limited - try again after in 55 seconds (more than 150, code: "api-deployments-paid-per-hour").

@gsoltis gsoltis force-pushed the gsoltis/large_monorepo branch from 17b3ead to 489e96e Compare February 15, 2022 00:33
@gsoltis gsoltis changed the base branch from gsoltis/e2e_benchmarks to main February 15, 2022 00:34
@gsoltis
Copy link
Contributor Author

gsoltis commented Feb 15, 2022

I am intending on replacing #721 with this PR.

@gsoltis gsoltis marked this pull request as ready for review February 15, 2022 01:08
@gsoltis gsoltis mentioned this pull request Feb 15, 2022
Closed
gaspar09
gaspar09 approved these changes Feb 15, 2022
View reviewed changes
benchmark/src/index.ts
Comment on lines +181 to +185
// Edit a dependency
const file = path.join(REPO_PATH, "apps", "navigation", "package.json");
const contents = JSON.parse(fs.readFileSync(file).toString("utf-8"));
contents.dependencies["crew-important-feature-0"] = "*";
fs.writeFileSync(file, JSON.stringify(contents, null, 2));
Copy link
Contributor

@gaspar09 gaspar09 Feb 15, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

Nit: Edit -> Add since crew-important-feature-0 is not originally a dependency of apps/navigation/package.json

@kodiakhq kodiakhq bot merged commit 4d42fa5 into main Feb 15, 2022
@kodiakhq kodiakhq bot deleted the gsoltis/large_monorepo branch February 15, 2022 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gaspar09 gaspar09 gaspar09 approved these changes

@jaredpalmer jaredpalmer Awaiting requested review from jaredpalmer

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gsoltis @gaspar09