Anyone can help review PRs

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Anyone can help review PRs

David Blevins-2
Just a reminder that anyone can help review PRs.  In fact, you'd likely be prioritized for getting commit access far sooner :)  Especially this month!

I made a small script to help pull down PRs : purl

 - https://gist.github.com/dblevins/6e255634d84914beb051a4b3858ebfb4

Just:
 
 cd tomee
 git checkout master
 git pull
 purl https://github.com/apache/tomee/pull/216

If it merges well, builds fine and you like the code, give the PR a thumbs up and maybe a little encouragement to the creator on the list.

--
David Blevins
http://twitter.com/dblevins
http://www.tomitribe.com

Reply | Threaded
Open this post in threaded view
|

Re: Anyone can help review PRs

Daniel Dias Dos Santos
very good this tips.
--

*Daniel Dias dos Santos*
Java Developer
SouJava & JCP Member
GitHub: https://github.com/Daniel-Dos
Linkedin: www.linkedin.com/in/danieldiasjava
Twitter: http://twitter.com/danieldiasjava


Em ter, 4 de dez de 2018 às 01:35, David Blevins <[hidden email]>
escreveu:

> Just a reminder that anyone can help review PRs.  In fact, you'd likely be
> prioritized for getting commit access far sooner :)  Especially this month!
>
> I made a small script to help pull down PRs : purl
>
>  - https://gist.github.com/dblevins/6e255634d84914beb051a4b3858ebfb4
>
> Just:
>
>  cd tomee
>  git checkout master
>  git pull
>  purl https://github.com/apache/tomee/pull/216
>
> If it merges well, builds fine and you like the code, give the PR a thumbs
> up and maybe a little encouragement to the creator on the list.
>
> --
> David Blevins
> http://twitter.com/dblevins
> http://www.tomitribe.com
>
>