making effective pull requests

asked Aug 6 '13

mat_pier
86 ●3

updated Oct 25 '0

sturkmen

6772 ●3 ●48 ●79 https://github.com/stu...

I've been using OpenCV over the past 3+ years. I've been an enthusiastic user, now trying to be a contributor to the project.

I'm also an newbie on Git. I know how to clone, fork, commit, push code to Github from my PC, edit code on Github. I panic when I have to do complex stuff (I just google in that case).

I have the following questions after doing pull request to the OpenCV project.

What do I do if pull requests shows that my code has certain warnings. Will that be fixed by the person by who is supposed to handle my pull request?
What do I do if there are errors. How to make a new request after fixing errors?

I both cases, do I delete my old fork, make a new fork of project, make changes again, make sure that there are no errors and request a new pull?

add a comment

2 answers

Sort by » oldest newest most voted

answered Aug 6 '13

berak
32993 ●7 ●81 ●312

updated Sep 1 '15

"Will that be fixed by the person by who is supposed to handle my pull request?"

no, your job. but you'll probably get some suggestions
"do I delete my old fork ?"

oh no, doing so will destroy all the history with it.just fix the problem, and update your fork(probably with -f option). the buildbot tests will be run automatically.
you want a separate branch for every problem you're trying to fix, look here

add a comment

answered Aug 6 '13

Mathieu Barnachon

4678 ●18 ●53 http://www.math-barnac...

According to the chart on that page, you will be ping by the PoD. So fix your errors and wait before resubmit that the PoD contacts you.

Comments

Let me add to this that you should sometimes have a bit of patience before a PR gets redacted and a PoD or follow-up is assigned. Many of the core devs have not enough time for the moment or are busy with other stuff, so a delay of a week is not abnormal.

StevenPuttemans (Sep 2 '15)edit

@StevenPuttemans I understand that during gsoc delay longer. Is it normal for this PR? I need a review about PR6664.

LBerger (Jul 1 '16)edit

Hello @LBerger, I am afraid we will have to be patient. I have PRs open from similar commit moments that are still waiting for final approval. I am guessing with Intel buying up Itseez, that it will only take a bit longer now :D

StevenPuttemans (Jul 1 '16)edit

add a comment

making effective pull requests

2 answers

Comments

Links

Question Tools

Stats

Related questions

making effective pull requests edit savecancel

2 answers

Comments

Links

Question Tools

Stats

Related questions

making effective pull requests