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

Skip to content

fix: make worker count int#213

Merged
vishnurk6247 merged 1 commit into
developfrom
fix/startup
Feb 5, 2026
Merged

fix: make worker count int#213
vishnurk6247 merged 1 commit into
developfrom
fix/startup

Conversation

@vishnurk6247

@vishnurk6247 vishnurk6247 commented Feb 5, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Bug Fixes
    • Corrected the production server worker configuration to properly convert the worker count parameter to an integer value. This ensures the application server initializes correctly with the specified number of worker processes, preventing type-handling issues that could arise when loading configuration from environment variables.

@coderabbitai

coderabbitai Bot commented Feb 5, 2026 •
edited
Loading

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Converted the UVicorn worker count parameter from a string to an integer by wrapping worker_count with int() in the production configuration path. This ensures the workers argument receives the correct numeric type when sourced from environment variables.

Changes

Cohort / File(s) Summary
Worker Count Type Conversion
wavefront/server/apps/floware/floware/server.py		 Wrapped worker_count with int() to convert string value to integer for uvicorn.run in production branch.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A string became a number, oh what joy!
In the production warren, no more type annoy,
Worker counts now integers, precise and bright,
The uvicorn server hops into the night! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/startup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vishnurk6247 vishnurk6247 merged commit 58d5032 into develop Feb 5, 2026
9 of 10 checks passed
@vishnurk6247 vishnurk6247 deleted the fix/startup branch February 5, 2026 11:33
thomastomy5 pushed a commit that referenced this pull request Apr 27, 2026
@vishnurk6247
fix: make worker count int (#213)
ca6b505
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rootflo-hardik rootflo-hardik rootflo-hardik 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.

2 participants

@vishnurk6247 @rootflo-hardik