[GitHub] opennlp-site issue #21: OPENNLP-1045: Add Git development page (adapted from...

2017-06-27 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/21 Happy to wait @kottmann and to amend the text as necessary. Not sure if others can edit this PR as this is in a branch in my fork, but if so, feel free to re-word the text as appropriate too. --

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Oleg Tikhonov
[x] +1 Migrate all repositories to GitHub On Tue, Jun 27, 2017 at 10:48 PM, Chris Mattmann wrote: > If you are talking about using Apache Gitbox, then yes I am +1 for this. > > Thanks, > Chris > > > > > On 6/27/17, 3:30 AM, "Joern Kottmann" wrote: > > Hello all, > > lets decide here i

[GitHub] opennlp-site issue #21: OPENNLP-1045: Add Git development page (adapted from...

2017-06-27 Thread chrismattmann
Github user chrismattmann commented on the issue: https://github.com/apache/opennlp-site/pull/21 something to consider once in Gitbox if the VOTE passes: http://wiki.apache.org/tika/UsingGit http://wiki.apache.org/nutch/UsingGit --- If your project is set up for it, you

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Chris Mattmann
If you are talking about using Apache Gitbox, then yes I am +1 for this. Thanks, Chris On 6/27/17, 3:30 AM, "Joern Kottmann" wrote: Hello all, lets decide here if we want to move our main repository, currently hosted at Apache to GitHub instead. This will make our process a

Some wisdom around Communities/Process/etc (was Re: [GitHub] opennlp pull request #238: Revert merging of sentiment work, no consent to m...)

2017-06-27 Thread Chris Mattmann
Hi Daniel, Thanks for your message. In actuality I was not really calling out your specific comment (and FWIW, we addressed it by getting the PR up to the same revision and not having 1000s of files modified, but just ~32 or so). So I wasn’t looking for any flattery there per-se. Please take the

[GitHub] opennlp-site pull request #21: OPENNLP-1045: Add Git development page (adapt...

2017-06-27 Thread jfrazee
Github user jfrazee commented on a diff in the pull request: https://github.com/apache/opennlp-site/pull/21#discussion_r124310132 --- Diff: src/main/jbake/content/using-git.ad --- @@ -0,0 +1,178 @@ + + Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] opennlp-site issue #21: OPENNLP-1045: Add Git development page (adapted from...

2017-06-27 Thread kottmann
Github user kottmann commented on the issue: https://github.com/apache/opennlp-site/pull/21 @kinow this will now change a bit again due to the migration to GitHub, I propose we hold it off another week. --- If your project is set up for it, you can reply to this email and have your

[GitHub] opennlp-site pull request #21: OPENNLP-1045: Add Git development page (adapt...

2017-06-27 Thread kottmann
Github user kottmann commented on a diff in the pull request: https://github.com/apache/opennlp-site/pull/21#discussion_r124306989 --- Diff: src/main/jbake/content/using-git.ad --- @@ -0,0 +1,178 @@ + + Licensed to the Apache Software Foundation (ASF) under one +

Re: [GitHub] opennlp pull request #238: Revert merging of sentiment work, no consent to m...

2017-06-27 Thread Dan Russ
Hi All, First, let me take a share of blame for the comment Chris mentioned. I believe I said something like the pull request was X revision behind and Y revisions ahead. It was not meant to be rude, it was meant to say it is hard to review code when it is so different from the current code

[GitHub] opennlp-site pull request #21: OPENNLP-1045: Add Git development page (adapt...

2017-06-27 Thread jfrazee
Github user jfrazee commented on a diff in the pull request: https://github.com/apache/opennlp-site/pull/21#discussion_r12431 --- Diff: src/main/jbake/content/using-git.ad --- @@ -0,0 +1,178 @@ + + Licensed to the Apache Software Foundation (ASF) under one +

Re: [GitHub] opennlp pull request #238: Revert merging of sentiment work, no consent to m...

2017-06-27 Thread Chris Mattmann
Hi everyone, I spoke with Joern in Slack. Some of his concerns are: 1. This was done with a Merge commit and apparently they squash and rebase. [would be helpful to see some pointer on this for documentation, thus far I haven’t found any] 2. Apparently we literally need to ask others for +1 vot

FW: [GitHub] opennlp pull request #238: Revert merging of sentiment work, no consent to m...

2017-06-27 Thread Chris Mattmann
Hi Joern, I’m confused. Why did you revert my commit? Every one of those check points you put on the PR was checked? We have been discussing this for months, you have seen the code for months, Ana and I have worked diligently on the code in plain view of everyone. Please explain. Chris On

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Anthony Beylerian
+1 On Tue, Jun 27, 2017 at 10:45 PM, Dan Russ wrote: > +1 > > > On Jun 27, 2017, at 9:28 AM, William Colen > wrote: > > > > +1 > > > > > > 2017-06-27 9:35 GMT-03:00 Suneel Marthi : > > > >> +1 > >> > >> मेरे iPhone से प्रेषित > >> > >> २७/०६/२०१७ को पू ८:२२ पर Jeff Zemerick ने लिखा : > >> > >>

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Dan Russ
+1 > On Jun 27, 2017, at 9:28 AM, William Colen wrote: > > +1 > > > 2017-06-27 9:35 GMT-03:00 Suneel Marthi : > >> +1 >> >> मेरे iPhone से प्रेषित >> >> २७/०६/२०१७ को पू ८:२२ पर Jeff Zemerick ने लिखा : >> >>> +1 >>> >>> On Tue, Jun 27, 2017 at 6:53 AM, Rodrigo Agerri >>> wrote: >>> >>>

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread William Colen
+1 2017-06-27 9:35 GMT-03:00 Suneel Marthi : > +1 > > मेरे iPhone से प्रेषित > > २७/०६/२०१७ को पू ८:२२ पर Jeff Zemerick ने लिखा : > > > +1 > > > > On Tue, Jun 27, 2017 at 6:53 AM, Rodrigo Agerri > > wrote: > > > >> +1 > >> > >> R > >> > >>> On Tue, Jun 27, 2017 at 12:46 PM, Mark G > wrote: >

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Suneel Marthi
+1 मेरे iPhone से प्रेषित २७/०६/२०१७ को पू ८:२२ पर Jeff Zemerick ने लिखा : > +1 > > On Tue, Jun 27, 2017 at 6:53 AM, Rodrigo Agerri > wrote: > >> +1 >> >> R >> >>> On Tue, Jun 27, 2017 at 12:46 PM, Mark G wrote: >>> >>> +1 >>> >>> Sent from my iPhone >>> On Jun 27, 2017, at 6:30 A

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Jeff Zemerick
+1 On Tue, Jun 27, 2017 at 6:53 AM, Rodrigo Agerri wrote: > +1 > > R > > On Tue, Jun 27, 2017 at 12:46 PM, Mark G wrote: > > > +1 > > > > Sent from my iPhone > > > > > On Jun 27, 2017, at 6:30 AM, Joern Kottmann > wrote: > > > > > > +1 > > > > > > Jörn > > > > > >> On Tue, Jun 27, 2017 at 12:3

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Rodrigo Agerri
+1 R On Tue, Jun 27, 2017 at 12:46 PM, Mark G wrote: > +1 > > Sent from my iPhone > > > On Jun 27, 2017, at 6:30 AM, Joern Kottmann wrote: > > > > +1 > > > > Jörn > > > >> On Tue, Jun 27, 2017 at 12:30 PM, Joern Kottmann > wrote: > >> Hello all, > >> > >> lets decide here if we want to move o

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Mark G
+1 Sent from my iPhone > On Jun 27, 2017, at 6:30 AM, Joern Kottmann wrote: > > +1 > > Jörn > >> On Tue, Jun 27, 2017 at 12:30 PM, Joern Kottmann wrote: >> Hello all, >> >> lets decide here if we want to move our main repository, currently >> hosted at Apache to GitHub instead. This will ma

[VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Joern Kottmann
Hello all, lets decide here if we want to move our main repository, currently hosted at Apache to GitHub instead. This will make our process a bit easier because we can eliminate one remote from our workflow. [ ] +1 Migrate all repositories to GitHub [ ] -1 Do not migrate, because...

Re: [VOTE] Migrate our main repositories to GitHub

2017-06-27 Thread Joern Kottmann
+1 Jörn On Tue, Jun 27, 2017 at 12:30 PM, Joern Kottmann wrote: > Hello all, > > lets decide here if we want to move our main repository, currently > hosted at Apache to GitHub instead. This will make our process a bit > easier because we can eliminate one remote from our workflow. > > [ ]

[GitHub] opennlp pull request #238: Revert merging of sentiment work, no consent to m...

2017-06-27 Thread kottmann
GitHub user kottmann opened a pull request: https://github.com/apache/opennlp/pull/238 Revert merging of sentiment work, no consent to merge it Thank you for contributing to Apache OpenNLP. In order to streamline the review of the contribution we ask you to ensure the fo