[feedhenry-dev] How are PRs to open source repos reviewed?

Martin Murphy mmurphy at redhat.com
Tue Nov 22 15:53:06 UTC 2016


Thanks Wojtek,
There are probably a few changes that I would suggest, I'll put them in a
PR.

Since there's no-one monitoring for PRs, an email to this list might make
sense when looking for a review.

Cheers,
M.

On 18 November 2016 at 16:35, Wojciech Trocki <wtrocki at redhat.com> wrote:

> We do not have any guide at the moment but..
> I'm trying to address this problem with this PR in help repo:
>
> https://github.com/fheng/help/pull/261
>
> Feel free to add some comments and your suggestions.
>
> Regards
>
> Wojtek T.
>
> On Thu, Nov 17, 2016 at 8:11 PM, Daniel Passos <dpassos at redhat.com> wrote:
>
>> There is an option on Github to receive an email for every new PR. I
>> think you can configure it for all repo or only for the repos you want.
>>
>> On Thu, Nov 17, 2016 at 6:05 PM, Martin Murphy <mmurphy at redhat.com>
>> wrote:
>>
>>> Thanks Passos.
>>>
>>> That would mean that someone on the team would have to be looking out
>>> for PRs to the open source repos.
>>> Do you know if we have that in place for all our open source repos?
>>>
>>> Cheers,
>>> M.
>>>
>>>
>>> On 17 November 2016 at 19:44, Daniel Passos <dpassos at redhat.com> wrote:
>>> > What we have been doing on AeroGear is:
>>> >
>>> > 1) If the PR came from the community?
>>> >
>>> > * Check is there is a Jira for this one, if not create one and link
>>> the PR
>>> > * Ping someone with more knowledgement about the project to review
>>> this one
>>> >
>>> > Example: https://github.com/aerogear/aerogear-ios-http/pull/73
>>> >                 https://github.com/aerogear/a
>>> erogear-ios-oauth2/pull/61
>>> >
>>> >
>>> > 2) If the PR came from the team
>>> >
>>> > * Add the Jira about the PR in the PR
>>> > * Ping someone with more knowledge about the project to review this one
>>> >
>>> > Example: https://github.com/aerogear/aerogear-android-auth/pull/37
>>> >
>>> > Bonus point to comment using reaction gifts[1]
>>> >
>>> > [1] http://www.reactiongifs.com/
>>> >
>>> >
>>> > On Thu, Nov 17, 2016 at 5:23 PM, Martin Murphy <mmurphy at redhat.com>
>>> wrote:
>>> >>
>>> >> How are PRs to open source repos reviewed?
>>> >> e.g. if someone submits a PR for fh-fhc, is that enough to get a
>>> review?
>>> >> Or do they have to ping someone/someplace?
>>> >>
>>> >> Cheers,
>>> >> M.
>>> >>
>>> >> _______________________________________________
>>> >> feedhenry-dev mailing list
>>> >> feedhenry-dev at redhat.com
>>> >> https://www.redhat.com/mailman/listinfo/feedhenry-dev
>>> >
>>> >
>>> >
>>> >
>>> > --
>>> > -- Passos
>>>
>>
>>
>>
>> --
>> -- Passos
>>
>> _______________________________________________
>> feedhenry-dev mailing list
>> feedhenry-dev at redhat.com
>> https://www.redhat.com/mailman/listinfo/feedhenry-dev
>>
>>
>
>
> --
> Wojciech Trocki
> Software Engineer, Red Hat Mobile
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/feedhenry-dev/attachments/20161122/b5700918/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: logo.png
Type: image/png
Size: 11472 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/feedhenry-dev/attachments/20161122/b5700918/attachment.png>


More information about the feedhenry-dev mailing list