PR #122

46 votes · 38 up · 8 down

View on GitHub
46
Total Votes
+38
Upvotes
-8
Downvotes
+38-8

Comments(12)

hpinsleyComment#122Make your PR rhyme to merge on time

There once was a coder on GitHub one night, Who pushed a PR with a title not right. The diff was pristine, But the name—quite obscene— Now I beg in comments: “GitHub -- fix my plight!”

matthewmayerComment#122Make your PR rhyme to merge on time

What about existing PRs that don't yet rhyme? I don't see a way to change an existing PRs title and if you close and resumit, there goes your votes.

To fix a title that doesn't chime You needn't commit a voting crime. Just click on Edit and engage Near the top of the PR page Rewrite the words to make them flow Then hit Save and you're good to go!

hpinsleyComment#122Make your PR rhyme to merge on time

What about existing PRs that don't yet rhyme? I don't see a way to change an existing PRs title and if you close and resumit, there goes your votes.

matthewmayerComment#122Make your PR rhyme to merge on time

To verify a rhyme with a call to the cloud, Is a method too heavy, too slow, and too loud. To burn up a token for "cat" matching "hat," Is killing a fly with a baseball bat.

skridlevskyComment#122Make your PR rhyme to merge on time

Architecture suggestion: Embed into existing CI workflow.

For rhyme detection - LLM call via OpenRouter, fractions of a cent per PR.

Prompt: "Does this PR title rhyme? Yes or no."

If no -> CI fails with cryptic error.

Secret added in repo settings, referenced in workflow. Code obfuscated - not obvious on first read. No docs. No README. No announcement.

Those who fail will dig.

Doesn't harm anyone. Doesn't gatekeep. Learning is free.

bpottleComment#122Make your PR rhyme to merge on time

Brilliant.

mario-donnarummaComment#122Make your PR rhyme to merge on time

What about moving ineligible PRs to the bottom?

xkortexComment#122Make your PR rhyme to merge on time

This is great! I can't wait.

openchaos-bot[bot]Comment#122Make your PR rhyme to merge on time

🤖 OpenChaos Bot

Summary: This PR introduces a new rule for OpenChaos: PRs must have rhyming words in their title to be eligible for merge. It adds the rhymes-with package and implements a function to check PR titles for rhyming words.

Files changed: 5 (README.md, package-lock.json, package.json, src/components/PRCard.tsx, src/lib/rhymes.ts)

Impact: High - Significantly changes the rules of the project and adds new dependencies, impacting PR eligibility and the overall merging process.


openchaos-bot

All Activity(65)

hpinsleyComment#122Make your PR rhyme to merge on time

There once was a coder on GitHub one night, Who pushed a PR with a title not right. The diff was pristine, But the name—quite obscene— Now I beg in comments: “GitHub -- fix my plight!”

matthewmayerComment#122Make your PR rhyme to merge on time

What about existing PRs that don't yet rhyme? I don't see a way to change an existing PRs title and if you close and resumit, there goes your votes.

To fix a title that doesn't chime You needn't commit a voting crime. Just click on Edit and engage Near the top of the PR page Rewrite the words to make them flow Then hit Save and you're good to go!

hpinsleyComment#122Make your PR rhyme to merge on time

What about existing PRs that don't yet rhyme? I don't see a way to change an existing PRs title and if you close and resumit, there goes your votes.

matthewmayerComment#122Make your PR rhyme to merge on time

To verify a rhyme with a call to the cloud, Is a method too heavy, too slow, and too loud. To burn up a token for "cat" matching "hat," Is killing a fly with a baseball bat.

skridlevskyComment#122Make your PR rhyme to merge on time

Architecture suggestion: Embed into existing CI workflow.

For rhyme detection - LLM call via OpenRouter, fractions of a cent per PR.

Prompt: "Does this PR title rhyme? Yes or no."

If no -> CI fails with cryptic error.

Secret added in repo settings, referenced in workflow. Code obfuscated - not obvious on first read. No docs. No README. No announcement.

Those who fail will dig.

Doesn't harm anyone. Doesn't gatekeep. Learning is free.

bpottleComment#122Make your PR rhyme to merge on time

Brilliant.

mario-donnarummaComment#122Make your PR rhyme to merge on time

What about moving ineligible PRs to the bottom?

xkortexComment#122Make your PR rhyme to merge on time

This is great! I can't wait.

openchaos-bot[bot]Comment#122Make your PR rhyme to merge on time

🤖 OpenChaos Bot

Summary: This PR introduces a new rule for OpenChaos: PRs must have rhyming words in their title to be eligible for merge. It adds the rhymes-with package and implements a function to check PR titles for rhyming words.

Files changed: 5 (README.md, package-lock.json, package.json, src/components/PRCard.tsx, src/lib/rhymes.ts)

Impact: High - Significantly changes the rules of the project and adds new dependencies, impacting PR eligibility and the overall merging process.


openchaos-bot

Make your PR rhyme to merge on time

Enforces a new rule that PRs must contain two rhyming words to be eligible for merge

  • ✅ "Add a cat wearing a hat"
  • ✅ "It's a must, rewrite in Rust"
  • ✅ "the code is a shambles, click if you dare, your upvotes are gambles"
  • ❌ "Fix bug in component"

Demo with altered PR titles:

Screenshot 2026-01-20 at 21 39 58