> It's good to usually merge stuff without waiting too long, particularly when 
> follow-up work is likely. In master those changes will likely get tested more 
> as more people will use them.

If there is a plan for a sequence of steps then that should be notified in an 
issue, possibly with a set of checkboxes listing the steps so they can be 
checked off as each PR on the process happens.  But you can't expect people to 
read your mind about what your future plans for follow up are.

Those of us here with English as a first language need to remember that 
although we have the same language, we don't have the same technical, 
professional, experience or cultural backgrounds.  So we may speak alike, but 
we do not think alike.  That means we need to put more effort into 
communicating our background reasoning and future intentions and not assume 
others will understand automatically and will come to the same conclusion about 
how a specific PR should be approached.

For instance, concern has been expressed that some PRs are [rearranging the 
deckchairs on the 
titanic](https://en.wiktionary.org/wiki/rearrange_the_deck_chairs_on_the_Titanic)
 and not achieving any substantive improvement.  Whilst I defend your right to 
align deckchairs, it would be good to know if their is a master plan behind it 
or just an itchy spot to scratch.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2270#issuecomment-526782250

Reply via email to