1. Mailing Lists
  2. Contributors
  3. Re: Proposal for new workflow, incorporating "Optimistic Merging"

Contributors mailing list archives

contributors@odoo-community.org

Browse archives

Re: Proposal for new workflow, incorporating "Optimistic Merging"

Re: Proposal for new workflow, incorporating "Optimistic Merging"

Avatar

Re: Proposal for new workflow, incorporating "Optimistic Merging"

by
Vauxoo, Nhomar Hernández
- 08/06/2016 00:20:45
Robert.

Some little enlights.

1.- If you want push without "Stupids +1" then simply push in your own repo, that's the deal (and I can not see more fear approach).
2.- I am blocked... even I have developments which I expend months to have them OCA compliant and that's ok (look for rma and you will enjoy), even me can NOT push freely without a proper +1 process, (and we contribute to core Pylint in python, PyGithub, Linux Core) then in every project I respect the minimal policies, give free PUSH acces is not open source that's Anarchy, and for that, create freely your repositories just "push and fuck governance".
3.- Win the right to Push it is not too difficult, I have members on my close friend environment with PUSH access to Plone repositories (yes Plone your own exmple) and they expend 3 years for such access, it is not as free as you describe.
4.- BTW as I always say, you can push to your own repo, declare your module in Odoo Apps and starts to contribute, or simply at least push in some place a zip with "whatever" you do, if the contribution is good PLEASE take the time to make a proper PR (it is not launch rockets to the moon) and no there are another tools in the world with such best power that git for such deal.

Regards.


On Tue, Jun 7, 2016 at 3:23 PM, robert rottermann <robert@redcor.ch> wrote:

Hi there

On 07/06/2016 19:08, Nhomar Hernandez wrote:
<blockquote cite="mid:CAKtu5Y7U+YWL8Kz0wEztg-=ABsbmeXAa8QBT0xit=Aa6y9B+xQ@mail.gmail.com" type="cite">

On Tue, Jun 7, 2016 at 11:53 AM, robert rottermann <robert@redcor.ch> wrote:

I believe we should try to find a way along this lines.
The actual situation is plain bad I think.
It actively discourages contributers and blocks good stuff waiting to be evaluated
EVERYONE of us should be able to check the code.

But EVERYONE needs a handle to judge what code is good and stable.


Hi Robert.

Thanks for the comment.
 de nada
<blockquote cite="mid:CAKtu5Y7U+YWL8Kz0wEztg-=ABsbmeXAa8QBT0xit=Aa6y9B+xQ@mail.gmail.com" type="cite">

Just FYI, now we (the commiters) only wait for 2 +1 (it is the rule) and EVERYONE can give a +1 in a PR, for responsability we ask make a real review but honestly that's based on trust some people say "+1 functional" or "+1 technical but I think x,y,z".... o not like it ..
It's not the way I work.
Or better said, this release workflow does not easily become a natural part of my development workflow.
If I need something I hunt for it, download and start to use it. Very often there is a need to interface with what I downloaded in some unique way. Thats when I start to learn its innards and stumble over problems. I probably improve little things ..

Nothing new here.  Thats how all of us work.

But what do I do with my findings (if any)?

I want to push it to the repo, so that the rest of the world (and me in some other setting) can easily use it.
I would not do it lightly, I might ask somebody to check it when it is more than cosmetics. I would make it pass all tests..
But I want it done and out of my mind in a very short time frame.

If this is not possible i fork and make it part of my own tool set...

<blockquote cite="mid:CAKtu5Y7U+YWL8Kz0wEztg-=ABsbmeXAa8QBT0xit=Aa6y9B+xQ@mail.gmail.com" type="cite">
We are more like commit channels (like the high level team side by side Linus Torvalds) even he say that honestly when he mention "I trust in no more than 10 persons, if they tested I merge" well that's more or less the same here.

You can easily Pick any PR (OCa member or not) just test it (functionally in runbot no technical skill needed) and give your +1.
 As said,  I want to use and fix (if necessary) I do not want to give any stupid +ses.
I hate all this millions of messages with nothing but a +.
If a tool I use would be marked with "please give feedback on .." I would do so
<blockquote cite="mid:CAKtu5Y7U+YWL8Kz0wEztg-=ABsbmeXAa8QBT0xit=Aa6y9B+xQ@mail.gmail.com" type="cite">

Then the fact that EVERYONE can judge is a matter of fact today, you +1 is respected and if you considered you tested partially something, just mention that than another person can give a +1 to the part you did not test.
 The only way I can test is using, adapt give back..
Of course, there should be well defined rules that control the process on how to change things. But they should allow that 90% of the pull request are resolved within a very short time.
<blockquote cite="mid:CAKtu5Y7U+YWL8Kz0wEztg-=ABsbmeXAa8QBT0xit=Aa6y9B+xQ@mail.gmail.com" type="cite">

I hope see you in github I am sure you will give .good visual of what you use, at least to giv feedback.

I think we must prepara a video with this working becuse the impression we are giving today is that we the mergers has the power of Block which is not totally true, we have the responsability of merge which is different.

Why no more Open, it means 100% have merge power?

We had that in this repository years ago: https://code.launchpad.net/openerp-extra

And the results were really BAD, the process we have today was to enable a minimal quality check because nobody was responsible at that time.
This I can easily believe. No RELEASE should be done without testing and approval.
But there should be a development repository where every OCA accredited developer is allowed to push.
And it should be sensibly easy to become an accredited developer.

<blockquote cite="mid:CAKtu5Y7U+YWL8Kz0wEztg-=ABsbmeXAa8QBT0xit=Aa6y9B+xQ@mail.gmail.com" type="cite">

Now you can know who is in chage of what, and you can (by yourself) simply give a ping if something is blocked and give you own +1 (even functionally).

I love the process we have now,
because you are NOT one of the many developers that are blocked ..

robert

_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe




--
Nhomar Hernandez
CEO Vauxoo.
Twitter: @nhomar
Odoo Gold Partner
Skype: nhomar00 (Envia mail previo no lo superviso siempre).
Móvil Venezuela:
+58 4144110269
Móvil México:
+52 1 4773933942

Reference