Re: Create patch instructions.

2015-08-07 Thread Artem Shutak
On Fri, Aug 7, 2015 at 12:14 AM, Konstantin Boudnik wrote: > On Wed, Aug 05, 2015 at 03:22PM, Artem Shutak wrote: > > Inline. > > > > On Wed, Aug 5, 2015 at 3:31 AM, Dmitriy Setrakyan > > > wrote: > > > > > Huge +1 on pull requests. Would be great if we could make TC work with > > > that. > > >

Re: Create patch instructions.

2015-08-06 Thread Konstantin Boudnik
On Wed, Aug 05, 2015 at 03:22PM, Artem Shutak wrote: > Inline. > > On Wed, Aug 5, 2015 at 3:31 AM, Dmitriy Setrakyan > wrote: > > > Huge +1 on pull requests. Would be great if we could make TC work with > > that. > > > > On Tue, Aug 4, 2015 at 5:30 PM, Alexey Kuznetsov > > wrote: > > > > > Arte

Re: Create patch instructions.

2015-08-05 Thread Yakov Zhdanov
inline --Yakov 2015-08-04 22:13 GMT+03:00 Artem Shutak : > Guys, > > I've updated > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute. > Please, take a look. > 1. I still come across few sprint-xx mentions 2. I would mention that contributors/committers can also fork GitHub m

Re: Create patch instructions.

2015-08-05 Thread Artem Shutak
Inline. On Wed, Aug 5, 2015 at 3:31 AM, Dmitriy Setrakyan wrote: > Huge +1 on pull requests. Would be great if we could make TC work with > that. > > On Tue, Aug 4, 2015 at 5:30 PM, Alexey Kuznetsov > wrote: > > > Artem, two comments: > > > > 1) Did you investigate possible workflow with "pull

Re: Create patch instructions.

2015-08-04 Thread Dmitriy Setrakyan
Huge +1 on pull requests. Would be great if we could make TC work with that. On Tue, Aug 4, 2015 at 5:30 PM, Alexey Kuznetsov wrote: > Artem, two comments: > > 1) Did you investigate possible workflow with "pull request" instead of > "patches"? > 2) In your instruction I see "merge" from master,

Re: Create patch instructions.

2015-08-04 Thread Alexey Kuznetsov
Artem, two comments: 1) Did you investigate possible workflow with "pull request" instead of "patches"? 2) In your instruction I see "merge" from master, but as I remember there was a discussion of using "rebase". What do you think? -- Alexey Kuznetsov GridGain Systems www.gridgain.com

Re: Create patch instructions.

2015-08-04 Thread Dmitriy Setrakyan
On Tue, Aug 4, 2015 at 12:13 PM, Artem Shutak wrote: > Guys, > > I've updated > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute. > Please, take a look. > Thanks! > Also, I think we can just remove > https://cwiki.apache.org/confluence/display/IGNITE/Sprint+Process. > Thoug

Re: Create patch instructions.

2015-08-04 Thread Artem Shutak
Guys, I've updated https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute. Please, take a look. Also, I think we can just remove https://cwiki.apache.org/confluence/display/IGNITE/Sprint+Process. Thoughts? >Why don't we have instructions on how to create a patch here? > http://ignit

Re: Create patch instructions.

2015-08-03 Thread Dmitriy Setrakyan
On Thu, Jul 30, 2015 at 11:52 AM, Sergi Vladykin wrote: > Brane, > > It appeared that I can't login into the wiki and for some reason it does > not send me email to reset my password.. Probably I have to file INFRA > issue? > Sergi, I just checked and you should have the sufficient permissions.

Re: Create patch instructions.

2015-08-03 Thread Konstantin Boudnik
Someone with admin rights on the space might be able to help you (unfortunately, I don't have such permissions). If not - try filing the INFRA ticket as you proposed. Cos On Thu, Jul 30, 2015 at 09:52PM, Sergi Vladykin wrote: > Brane, > > It appeared that I can't login into the wiki and for some

Re: Create patch instructions.

2015-07-30 Thread Sergi Vladykin
Brane, It appeared that I can't login into the wiki and for some reason it does not send me email to reset my password.. Probably I have to file INFRA issue? Sergi 2015-07-30 14:43 GMT+03:00 Branko Čibej : > Sergi, why don't you just fix the Wiki? > > -- Brane > > On 30.07.2015 11:18, Sergi Vla

Re: Create patch instructions.

2015-07-30 Thread Branko Čibej
Sergi, why don't you just fix the Wiki? -- Brane On 30.07.2015 11:18, Sergi Vladykin wrote: > I meant this wiki page: > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > Sergi > > 2015-07-30 12:17 GMT+03:00 Sergi Vladykin : > >> Also a wiki page should contain instructions

Re: Create patch instructions.

2015-07-30 Thread Sergi Vladykin
I meant this wiki page: https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute Sergi 2015-07-30 12:17 GMT+03:00 Sergi Vladykin : > Also a wiki page should contain instructions how to provide patch for > release branch instead of master. > > By the way since we have these patching s

Re: Create patch instructions.

2015-07-30 Thread Sergi Vladykin
Also a wiki page should contain instructions how to provide patch for release branch instead of master. By the way since we have these patching scripts only for bash, we probably should provide additional instruction for Windows users. Sergi P.S. Just trying to contribute something to Ignite and

Re: Create patch instructions.

2015-07-30 Thread Sergi Vladykin
Also the page contains outdated instructions about sprints: *The "master" branch contains the last released version. The "sprint-N" branch contains the latest code for the Nth sprint* Sergi 2015-07-30 12:05 GMT+03:00 Sergi Vladykin : > Guys, > > Why don't we have instructions on how to c