[GitHub] opennlp pull request #147: Fix typos in NER doc, and in package.html file

2017-03-26 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp/pull/147 Fix typos in NER doc, and in package.html file Found a couple more typos. The first one was right in front of me when I submitted another PR these days. The other was in a package HTML doc. You can

[GitHub] opennlp pull request #146: Fix typo

2017-03-25 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp/pull/146 Fix typo Hi, fixing trivial typo (s/aditiona/additional) in the NER docs. Cheers Bruno You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] opennlp-site pull request #23: OPENNLP-1103 add AirNZ case

2017-06-29 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/23 OPENNLP-1103 add AirNZ case First case from AirNZ. See ticket for more background on where it came from. The following screen shot displays what the website looks like after applying

[GitHub] opennlp-site pull request #24: OPENNLP-1104 fix poweredby images, and use lo...

2017-06-29 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/24 OPENNLP-1104 fix poweredby images, and use lower case in url's Fixes the images at the bottom of the page, and uses lower case to match what other pages look like. You can merge this pull

[GitHub] opennlp-site issue #24: OPENNLP-1104 fix poweredby images, and use lower cas...

2017-06-29 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/24 ![after](https://user-images.githubusercontent.com/304786/27715495-aaa2999c-5d8b-11e7-8dd8-66450a299cf0.png) --- If your project is set up for it, you can reply to this email and have your

[GitHub] opennlp-site issue #24: OPENNLP-1104 fix poweredby images, and use lower cas...

2017-06-29 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/24 ![before](https://user-images.githubusercontent.com/304786/27715491-a5f6f1e0-5d8b-11e7-91b1-f32db66c2d7f.png) --- If your project is set up for it, you can reply to this email and have your

[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

[GitHub] opennlp-site pull request #5: OPENNLP-393: add contributors wanted issues li...

2017-05-10 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/5 OPENNLP-393: add contributors wanted issues list As per OPENNLP-393 comments, updating the Get Involved page with the list of contributions wanted issues. Screen shot of what

[GitHub] opennlp-site pull request #9: Fix RSS feed title, description, and link

2017-05-11 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/9 Fix RSS feed title, description, and link Our RSS feed is displaying the title as JBake, as well as description and the link too. This pull request fixes all these items. You can merge this pull

[GitHub] opennlp-site pull request #8: OPENNLP-504 adding initial pairs of question a...

2017-05-11 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/8 OPENNLP-504 adding initial pairs of question and answers to the FAQ This is a pull request for [OPENNLP-504](https://issues.apache.org/jira/browse/OPENNLP-504), including the initial frequently

[GitHub] opennlp-site issue #6: OPENNLP-1053: Adds doap_opennlp.rdf.

2017-05-11 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/6 Merged. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] opennlp-site issue #7: Remove Google Analytics tracking code and JS code

2017-05-11 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/7 Merged. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] opennlp-site pull request #7: Remove Google Analytics tracking code and JS c...

2017-05-11 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/7 Remove Google Analytics tracking code and JS code This pull request removes the Google Analytics tracking code and its accompanying JavaScript code. You can merge this pull request into a Git

[GitHub] opennlp-site issue #5: OPENNLP-393: add contributors wanted issues list

2017-05-11 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/5 I suggested something like that, but @kottmann raised a good point that JIRA is not that reliable, and could cause issues in the web site. Also, we would stress a bit more on our JIRA server. Now

[GitHub] opennlp-site issue #8: OPENNLP-504 adding initial pairs of question and answ...

2017-05-11 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/8 ![opennlp-faq-wip-20170512-fullpage](https://cloud.githubusercontent.com/assets/304786/25949773/d111416a-36ac-11e7-8954-83bbb66de83c.png) Screenshot with the questions. Instead of using

[GitHub] opennlp-site pull request #10: Avoid using a HTTP CDN to download FontAwesom...

2017-05-18 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/10 Avoid using a HTTP CDN to download FontAwesome, use it from the server instead This pull request addresses error messages found in the console when browsing https://opennlp.apache.org

[GitHub] opennlp-site issue #4: Adds News page.

2017-05-18 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/4 @jzonthemtn this has been merged, but the ASF bot - for some reason - is not closing pull requests merged (even with the "This closes #1234" commit message). Could you close

[GitHub] opennlp-site pull request #9: Fix RSS feed title, description, and link

2017-05-18 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/opennlp-site/pull/9 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] opennlp-site issue #9: Fix RSS feed title, description, and link

2017-05-18 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/9 Merged in master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] opennlp-site pull request #11: OPENNLP-1045: Git documentation for developer...

2017-05-18 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/11 OPENNLP-1045: Git documentation for developers Add documentation for development with Git (at ASF, GitHub, etc) for OpenNLP. Inspired mainly at the Cordova docs [1]. Did not include

[GitHub] opennlp pull request #229: OPENNLP-1091 Findbugs issues and IDE warnings

2017-06-09 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp/pull/229 OPENNLP-1091 Findbugs issues and IDE warnings Thank you for contributing to Apache OpenNLP. In order to streamline the review of the contribution we ask you to ensure the following steps

[GitHub] opennlp-site pull request #11: OPENNLP-1045: Git documentation for developer...

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

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

2017-06-22 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/21 OPENNLP-1045: Add Git development page (adapted from Mahout) This is an alternative to #11, which follows the suggestion from @smarthi, and adapts the page from the Mahout page on Git and Pull

[GitHub] opennlp-site pull request #14: OPENNLP-1067: Use variables in jbake.properti...

2017-05-19 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/14 OPENNLP-1067: Use variables in jbake.properties file to specify version used in site generation. We cannot use pom.xml, as the feature in the JBake maven plug-in was released with the jbake

[GitHub] opennlp-site pull request #14: OPENNLP-1067: Use variables in jbake.properti...

2017-05-19 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/opennlp-site/pull/14#discussion_r117456370 --- Diff: src/main/jbake/content/maven-dependency.ad --- @@ -35,59 +34,75 @@ all transient dependencies are resolved automatically. To use

[GitHub] opennlp-site pull request #14: OPENNLP-1067: Use variables in jbake.properti...

2017-05-19 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/opennlp-site/pull/14#discussion_r117456563 --- Diff: src/main/jbake/content/maven-dependency.ad --- @@ -35,59 +34,75 @@ all transient dependencies are resolved automatically. To use

[GitHub] opennlp-site issue #13: Update download links to 1.8.0

2017-05-19 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/13 +1 lgtm --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] opennlp-site pull request #14: OPENNLP-1067: Use variables in jbake.properti...

2017-05-19 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/opennlp-site/pull/14#discussion_r117456032 --- Diff: src/main/jbake/content/docs/index.ad --- @@ -25,12 +25,12 @@ There exists a manual and Javadoc API documentation for Apache OpenNLP

[GitHub] opennlp-site pull request #10: Avoid using a HTTP CDN to download FontAwesom...

2017-05-18 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/opennlp-site/pull/10 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] opennlp-site issue #10: Avoid using a HTTP CDN to download FontAwesome, use ...

2017-05-18 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/10 Merged in 7e7e79b4374458a38bce5a1b752473b4eb326f7a Closing manually. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] opennlp-site issue #4: Adds News page.

2017-05-18 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/4 @smarthi this has been merged already, but we need either @jzonthemtn to close the pull request, or wait till someone makes master the default branch in this repository. Then we can close it via

[GitHub] opennlp-site issue #12: Update site for 1.8.0 release

2017-05-18 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/12 Maybe we should update the Documentation page too (docs/index.ad)? Other than that, +1 lgtm --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] opennlp-site pull request #20: OPENNLP-1093: bump up version of jbake-maven-...

2017-06-13 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/20 OPENNLP-1093: bump up version of jbake-maven-plugin, and update groupId Tested locally both running the site, and generating the files during the package version. You can merge this pull

[GitHub] opennlp-site issue #11: OPENNLP-1045: Git documentation for developers

2017-06-13 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/11 @smarthi I remember you had some Git docs from Mahout (I think) that you thought we should use too. Are you happy too with us using this version in this pull request for now, and then start

[GitHub] opennlp-site pull request #20: OPENNLP-1093: bump up version of jbake-maven-...

2017-06-15 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/opennlp-site/pull/20 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] opennlp-site issue #20: OPENNLP-1093: bump up version of jbake-maven-plugin,...

2017-06-15 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/20 Merged. Closing manually as integration with GitHub still not working (need to make master the principal branch) --- If your project is set up for it, you can reply to this email and have your

[GitHub] opennlp-site pull request #1: OPENNLP-999: new website layout with JBake

2017-05-05 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/1 OPENNLP-999: new website layout with JBake New website layout with JBake. The diff is too big, due to the deleted files. Might be easier to compare by checking out this pull request locally. You

[GitHub] opennlp pull request #182: OPENNLP-999: Use JBake for website (thanks to @wc...

2017-05-04 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/opennlp/pull/182 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] opennlp-site issue #3: Adds News page.

2017-05-08 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/3 @jzonthemtn I think we should use this approach here: https://github.com/wcolen/opennlp/blob/opennlp-site/opennlp-site/src/main/jbake/templates/news.ftl Would you like/have time to give

[GitHub] opennlp-site issue #1: OPENNLP-999: new website layout with JBake

2017-05-06 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/1 Closing as we are going instead with approach in pull request #2 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] opennlp-site pull request #1: OPENNLP-999: new website layout with JBake

2017-05-06 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/opennlp-site/pull/1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] opennlp-site pull request #2: OPENNLP-999: new website layout with JBake

2017-05-06 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/opennlp-site/pull/2 OPENNLP-999: new website layout with JBake New website layout with JBake. This replaces pull request #1, but instead of having a diff against old content, we have a newly created - and empty

[GitHub] opennlp-site issue #4: Adds News page.

2017-05-08 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/4 +1! Tested locally, and worked fine. Thought it was missing the template file, but it was actually the index under the news folder. Nice catch. --- If your project is set up for it, you can reply

[GitHub] opennlp-site pull request #14: OPENNLP-1067: Use variables in jbake.properti...

2017-05-21 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/opennlp-site/pull/14 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] opennlp-site issue #14: OPENNLP-1067: Use variables in jbake.properties file...

2017-05-21 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/opennlp-site/pull/14 Merged in 15ebab57cc56e9c795282892f695211a9d703c5e --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have