Found myself nodding along. I think increasingly it's useful to think of PRs from unknown external people as more like an issue than a PR (kind of like the 'issue first' policy described in the article).

There's actually something very valuable about a user specifying what they want using a working solution, even if the code is not mergeable.

Writing code was always the easy part!

Getting code to work though..