PR #194

Should also do the check in the automated daily merge action.

Good point, thanks.

Should also do the check in the automated daily merge action.

A video's fine, it's pictures in a line. Upload away, your PR will stay.

When the world's in disarray, would a video be okay?

The picture rule is merged and clean, the simplest check you've ever seen. No pic? No merge. That's internet law — enforced at last, without a flaw.

The stray files purged, the logic tight, @matthewmayer set us right. Now every PR, big or small, must drag and drop or face the wall.

fin.

this and a few other files were likely added by mistake in this PR

this can be refactored to make it more DRY and avoid a combinatorial explosion i.e.

const issues = [];

if (!pr.isMergeable) issues.push("Merge conflicts");
if (!containsRhymes) issues.push("No rhyme");
if (!pr.hasPicture) issues.push("No pic");
if (!pr.checksPassed) issues.push("Checks failed");

🤖 OpenChaos Bot

Summary: This PR adds a new rule to OpenChaos requiring PRs to include a picture uploaded directly to GitHub in order to be eligible for merging. It also updates the UI to reflect this new rule.

Files changed: 4 (README.md, src/components/PRCard.tsx, src/lib/github.ts, src/lib/pictures.ts)

Vibe: This PR is adding fuel to the fire of chaos, as if internet democracy needed more arbitrary rules.

openchaos-bot