Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mkuthan
Copy link
Member

@mkuthan mkuthan commented Jul 16, 2025

No description provided.

@mkuthan mkuthan requested a review from a team as a code owner July 16, 2025 14:52
Copy link
Contributor

@piter75 piter75 left a comment

Choose a reason for hiding this comment

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

LGTM

cluster,
verbose: this.verbose,
concurrentLimit: 5,
concurrentLimit: cluster.concurrentLimit || 5,
Copy link
Contributor

Choose a reason for hiding this comment

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

😂

Choose a reason for hiding this comment

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

🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

if someone sets limit to 0, fallback to 5 looks fine :D

@github-actions
Copy link

⏱️ Lighthouse report

@github-actions
Copy link

size-limit report 📦

Path Size
build/public/main.js 2.29 MB (0%)
build/public/polyfills.es5.js 42.7 KB (0%)
build/public/dnd.es5.js 4.61 KB (0%)

@github-actions
Copy link

✅ Deployed successfully to: https://turnilo-concurrency-limit-config-gmbbyye42a-ew.a.run.app

@mkuthan mkuthan merged commit f840ef0 into master Jul 17, 2025
13 checks passed
@mkuthan mkuthan deleted the concurrency_limit_config branch July 17, 2025 05:50
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.

4 participants