A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://github.com/crankyoldgit/IRremoteESP8266/issues/115 below:

Looking for people to do code reviews. · Issue #115 · crankyoldgit/IRremoteESP8266 · GitHub

Hi,
Do you know how to program? Are familiar with C/C++ or other languages (python/go/etc)? If so, you may be able to help us.

Under the principle that many & different eyes might catch mistakes that a code author might miss, I'm after people to review all of the code prior to commits. Ideally, we'd have more than one person review a commit, but starting with at least one is a first step.

What the job of a code reviewer entails is someone to check if the code looks sane, would be similar to how they would implement it, check for any obvious boo-boos, offer advice, and offer constructive criticism. A code review(er) is not responsible for making sure there are no bugs, or anything. They are never to blame if something goes wrong. All responsibility is with the code author basically. Not you. So there is little pressure.

Currently @roidayan (mostly) & @markszabo are doing a lot of the reviews (and myself vice-versa) and I feel bad hitting them up for every change I make. So, it'd be real nice to them to if you'd like to sign up so we can spread the workload around.

If you feel like you can do this, drop a comment on this issue indicating that, or if you have further questions about the task.

Thanks.


RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4