PR #122

28 votes · 23 up · 5 down

View on GitHub
28
Total Votes
+23
Upvotes
-5
Downvotes
+23-5

Comments(7)

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(39)

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