Hmmm...
Well, first of all thanks for providing some guidelines. By the sound of it, I messed up. In the sense that
my very first PR, which I issued only a few days ago, is a perfect mix of scenario 1 (small changes, documentation) and scenario 3 (with questions). Immortius saw right through it though and closed it without merging it in, although he was kind enough to answer to each issue.
Ok, I need to understand a bit better how my workflow should be in this situation. What I did to get to the PR mentioned above is to go through files, add comments, change small things and add some questions. At the end I therefore found myself with an heterogeneous number of changes that really needed to be on two branches: one for added comments and small changes and the other for questions. So, my question is: should I, from this scenario, proceed to manually split the changes onto two branches, or should I have not even arrived at this scenario, i.e. by disciplining myself to only focus on asking questions in one branch and do changes in another?
Also, how big should a PR be? Is it common to have one PR per file or even per change?
Thanks again for your help!