badges Pull Request Reviews?

Ralph Bean rbean at redhat.com
Mon Aug 19 16:25:27 UTC 2013


Hi all,

If anyone has some spare cycles, I could use some pull request
reviews.

To review a pull request, you don't need any special credentials.
Just give the code changes a read-over.  If anything is confusing or
you don't know why the change was made, please ask.  It will help to
clarify for everyone.

These four are independent:

- https://github.com/fedora-infra/tahrir/pull/201
- https://github.com/fedora-infra/tahrir/pull/200
- https://github.com/fedora-infra/tahrir/pull/194
- https://github.com/fedora-infra/fedbadges/pull/22

These three are interrelated, starting with the core one first:

- https://github.com/fedora-infra/tahrir-api/pull/19
- https://github.com/fedora-infra/fedbadges/pull/23
- https://github.com/fedora-infra/tahrir/pull/199

We do code reviews in #fedora-apps to try and catch bugs early before
they make it into production.

Cheers, and thanks in advance-
 -Ralph
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.fedoraproject.org/pipermail/badges/attachments/20130819/e78f1d7a/attachment.sig>


More information about the badges mailing list