Hacker News Reader

Parent Full thread

paxys

Not at all. It isn't the size of the PR that matters, but I generally want contributors to:

- Discuss their changes beforehand (usually on the issue page)

- Use patterns and practices that the rest of the codebase follows

- Listen to comments and feedback

- Write tests

While the work is "free", it should still be treated professionally.

View on HN

> - Discuss their changes beforehand (usually on the issue page)

Interesting. When I last looked at contributing to open source, some general advice was to offer up code even if its likely the wrong approach. The idea being that maintainers are busy, and there's lots of people that drag on their time without ever contributing anything. Showing you're actually willing to put in time and effort will make the maintainer more likely to guide you in the correct direction because you've already shown you're willing to spend time on the issue.

I feel the same way. Although I have only been starting to contribute recently, I thought that by offering code first via pull request I wouldn't be perceived as a whiner in the issues section.

This is definitely true. I'm not a fan of Issues discussion with random people who are not known to be reliable contributors. Talk is cheap, show me the code.

Reply on news.ycombinator.com