Skip to content

env: use available concurrency to set concurrency defaults #104

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
dylanlott merged 2 commits into from
Jun 16, 2025
Merged

env: use available concurrency to set concurrency defaults #104

dylanlott merged 2 commits into from
Jun 16, 2025

Conversation

@dylanlott
Copy link
Contributor

@dylanlott dylanlott commented Jun 12, 2025

No description provided.

This was referenced Jun 12, 2025
Merged
Merged
Merged
Merged
@dylanlott Graphite App
Copy link
Contributor Author

dylanlott commented Jun 12, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dylanlott dylanlott mentioned this pull request Jun 12, 2025
Merged
prestwich
prestwich reviewed Jun 13, 2025
View reviewed changes
@prestwich prestwich marked this pull request as ready for review June 13, 2025 13:29
@prestwich prestwich requested a review from Evalir as a code owner June 13, 2025 13:29
Evalir
Evalir reviewed Jun 13, 2025
View reviewed changes
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 97d070d to f33dd1d Compare June 13, 2025 21:17
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 2d635b1 to 3bfa9d0 Compare June 13, 2025 21:17
@dylanlott dylanlott force-pushed the dylan/update_readme branch from f33dd1d to 58fe168 Compare June 13, 2025 21:20
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 3bfa9d0 to 211d6d6 Compare June 13, 2025 21:20
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 58fe168 to 616215e Compare June 13, 2025 22:16
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 211d6d6 to 06d3de1 Compare June 13, 2025 22:16
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 616215e to 8cd1cf5 Compare June 13, 2025 22:20
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 06d3de1 to bd2d48d Compare June 13, 2025 22:20
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 8cd1cf5 to db40995 Compare June 13, 2025 23:56
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from bd2d48d to 14965e1 Compare June 13, 2025 23:56
This was referenced Jun 13, 2025
Closed
Merged
Merged
@dylanlott dylanlott marked this pull request as draft June 13, 2025 23:59
@dylanlott dylanlott force-pushed the dylan/update_readme branch from db40995 to a6d000e Compare June 14, 2025 00:10
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 14965e1 to 6fda7c1 Compare June 14, 2025 00:10
@dylanlott dylanlott force-pushed the dylan/update_readme branch from a6d000e to 244b0a4 Compare June 14, 2025 00:13
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch 2 times, most recently from f134c9b to 8bbf390 Compare June 14, 2025 00:15
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 50b3154 to b39ef8a Compare June 14, 2025 00:20
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch 2 times, most recently from ea9dc58 to 8da528c Compare June 14, 2025 00:41
@dylanlott dylanlott force-pushed the dylan/update_readme branch from dbef65a to 241c18a Compare June 14, 2025 02:35
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 0416049 to 656752c Compare June 14, 2025 02:35
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 241c18a to cceab25 Compare June 14, 2025 02:36
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 656752c to 24a725e Compare June 14, 2025 02:36
@dylanlott dylanlott marked this pull request as ready for review June 14, 2025 02:44
@dylanlott dylanlott mentioned this pull request Jun 14, 2025
Merged
@Evalir Evalir mentioned this pull request Jun 16, 2025
Merged
@dylanlott dylanlott force-pushed the dylan/update_readme branch from cceab25 to 28df5a9 Compare June 16, 2025 16:20
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 24a725e to 4a1ed69 Compare June 16, 2025 16:20
@Evalir Evalir mentioned this pull request Jun 16, 2025
Merged
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 28df5a9 to 3176e63 Compare June 16, 2025 17:06
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 4a1ed69 to 804c1ee Compare June 16, 2025 17:06
@dylanlott dylanlott force-pushed the dylan/update_readme branch from 3176e63 to 8d50ae1 Compare June 16, 2025 17:10
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 804c1ee to 87e3b9a Compare June 16, 2025 17:10
@dylanlott dylanlott force-pushed the dylan/update_readme branch 2 times, most recently from b582b23 to 3fd866a Compare June 16, 2025 17:17
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 87e3b9a to 9bb6f01 Compare June 16, 2025 17:18
@dylanlott dylanlott changed the base branch from dylan/update_readme to graphite-base/104 June 16, 2025 17:23
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 9bb6f01 to 42413b3 Compare June 16, 2025 17:23
@dylanlott dylanlott force-pushed the graphite-base/104 branch from 3fd866a to 59d1fc9 Compare June 16, 2025 17:23
@graphite-app graphite-app bot changed the base branch from graphite-base/104 to main June 16, 2025 17:24
@dylanlott
memoizes the available parallelism call
ad5a1a9 
- bump default concurrency const to 8 from 4
- memoize the result of the available parallelism call
- make `concurrency_limit` config property into `Option<usize>` instead
of `usize`
@dylanlott dylanlott force-pushed the dylan/env-available-concurrency branch from 42413b3 to ad5a1a9 Compare June 16, 2025 17:24
@dylanlott Graphite App
Copy link
Contributor Author

dylanlott commented Jun 16, 2025 •
edited
Loading

Merge activity

  • Jun 16, 5:27 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 16, 5:27 PM UTC: @dylanlott merged this pull request with Graphite.

@dylanlott dylanlott merged commit ae37a40 into main Jun 16, 2025
6 checks passed
@dylanlott dylanlott deleted the dylan/env-available-concurrency branch June 16, 2025 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@prestwich prestwich prestwich left review comments

@Evalir Evalir Evalir approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dylanlott @prestwich @Evalir