Hello Dieter, You are the "component owner" so you can commit directly in case "you feel lucky (c)" and don't need a review PR is necessary in case you need second opinion or two :)
On Fri, Dec 8, 2017 at 2:16 PM, Dieter Tremel <[email protected]> wrote: > Hello wicketstuff team, > > as newbie wicketstuff contributor (gchart) I have to ask some questions > around team workflow. > > Sorry if my knowledge here is not sufficient, I work mostly alone for a > long time, so I had to improve my git and github handling a lot. And of > course I don't want initiate any silly or destructive actions on the > github repository. > > I have improved gchart in 5 commits and this should be moved to the > repository. This is my setup including my fork: > >> $ git remote -v >> origin https://[email protected]/tremel/core.git (fetch) >> origin https://[email protected]/tremel/core.git (push) >> upstream https://[email protected]/wicketstuff/core.git (fetch) >> upstream https://[email protected]/wicketstuff/core.git (push) > > I already pulled and merged the actual master from wicketstuff/core and > pushed to tremel/core. So I could easily start a pull request. > > My commits are in a branch named gchart-ajax-fix, pushed to tremel/core > too. As far as I understand, as a contributor I could commit directly to > wicketstuff/core/gchart ("no merge commits") but also start a pull > request from tremel/core. What is the right way to to it? > > I suppose small commits like typos in gchart I should do without > disturbing you with a pull request, larger improvements you should look > at by pull request from tremel/core. Right? > > Thank you for some advice. > Dieter > -- WBR Maxim aka solomax
