Remix.run Logo
whateverboat 8 hours ago

This was the original rule in linux kernel as well. No more than 200 loc per patch. We should also introduce this to git commits and pull request descriptions:

1. 400 chars/10 lines per commit

1b. Not more than 3 commits in the initial pull request

2. 20 lines of explanation for pull request

3. not more than 3 pull request open at any one time

TomasBM 6 hours ago | parent [-]

Seems like this policy would apply pretty well regardless of who/what generated the code.

mexicocitinluez 5 hours ago | parent [-]

YES! "No AI" policies that are purely based on technical grounds make no sense to me. Bad PR's are bad PR's regardless of their source.

Are we really in a situation where good code that solves a problem won't be merged because the person the person checked the "I used AI" box on the PR?

Ban PR's that are too big, don't have a clear purpose, touch too many areas, etc.

overgard 27 minutes ago | parent [-]

It's really a question of how much time you're willing to spend sorting through spam. "No AI" might be a blunt hammer, but the people submitting slop aren't reading guidelines anyway, and it's easier just to reject it early. Frankly, I'm sure if people wanted to sneak in an AI generated code by carefully reviewing it and making sure it's targeted and well tested... I'm sure they could, but those people aren't the problem.