Skip to content

Add careers link to README and startup log#13897

Merged
alexisrolland merged 2 commits into
masterfrom
hiring-link
May 14, 2026
Merged

Add careers link to README and startup log#13897
alexisrolland merged 2 commits into
masterfrom
hiring-link

Conversation

@robinjhuang
Copy link
Copy Markdown
Member

@robinjhuang robinjhuang commented May 14, 2026

Adds a small link to comfy.org/careers in the README and a one-line note in the server startup output.

Screenshot 2026-05-14 at 10 10 06 AM

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 14, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This PR adds hiring promotion across two surfaces. The README gains a hiring callout in the "Support and dev channel" section, directing readers to the careers page. The server startup now logs the same hiring message when running in verbose mode, appearing alongside the initial "Starting server" output. Both additions reference comfy.org/careers.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: adding a careers link to both the README and server startup log.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The PR description clearly relates to the changeset, describing the addition of a careers link to the README and server startup log.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

@alexisrolland alexisrolland merged commit 3f9bdc7 into master May 14, 2026
21 checks passed
@comfyanonymous comfyanonymous deleted the hiring-link branch May 14, 2026 19:24
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.

3 participants