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

[ci skip] Indicate that wait_for_less_busy_worker is enabled by default in DSL docs #3371

Conversation

joshuay03
Copy link
Contributor

@joshuay03 joshuay03 commented Apr 16, 2024

Description

wait_for_less_busy_worker was enabled by default with #2940 and included in the 6.0.0 release. The DSL docs do not reflect this however unlike other config options that are enabled by default.

Your checklist for this pull request

  • I have reviewed the guidelines for contributing to this repository.
  • I have added (or updated) appropriate tests if this PR fixes a bug or adds a feature.
  • My pull request is 100 lines added/removed or less so that it can be easily reviewed.
  • If this PR doesn't need tests (docs change), I added [ci skip] to the title of the PR.
  • If this closes any issues, I have added "Closes #issue" to the PR description or my commit messages.
  • I have updated the documentation accordingly.
  • All new and existing tests passed, including Rubocop.

@joshuay03 joshuay03 force-pushed the mention-wait-for-less-busy-worker-enabled-by-default branch from 3204d64 to 41ea8b7 Compare April 16, 2024 10:51
@joshuay03 joshuay03 changed the title [ci skip] Indicate that wait_for_less_busy_worker is enabled by default in DSL docs [ci skip] Indicate that wait_for_less_busy_worker is enabled by default in DSL docs Apr 16, 2024
@dentarg dentarg merged commit 7a999de into puma:master Apr 16, 2024
@joshuay03
Copy link
Contributor Author

Thank you @dentarg

@joshuay03 joshuay03 deleted the mention-wait-for-less-busy-worker-enabled-by-default branch April 16, 2024 11:44
rus-max pushed a commit to rus-max/puma that referenced this pull request Apr 25, 2024
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.

None yet

2 participants