Github user dszeto commented on the issue:
https://github.com/apache/incubator-predictionio-sdk-ruby/pull/22
I am going to merge this and make a new release. Thanks @anothermh !
---
Github user dszeto commented on a diff in the pull request:
https://github.com/apache/incubator-predictionio-sdk-php/pull/24#discussion_r147047179
--- Diff: README.md ---
@@ -42,23 +42,11 @@ By Composer
The easiest way to install PredictionIO PHP client is to use
Github user dszeto commented on the issue:
https://github.com/apache/predictionio-sdk-php/pull/26
Your change was included in 0.9.1 that was released yesterday.
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio-sdk-php/pull/24
Sorry for the delay. This looks good to me now. Thanks @cafferata !
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio-sdk-php/pull/26
LGTM. THanks @sgrodzicki !
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/450
@voran Thank you for your contribution! Would you please fix the license
check test, file a ticket under Apache JIRA
(https://issues.apache.org/jira/projects/PIO/issues) and change the subject
Github user dszeto commented on the pull request:
https://github.com/apache/predictionio/commit/e734ff89db03d5a2acc409dc7e4ccb0cc37a1065#commitcomment-27940360
In docs/manual/source/install/install-sourcecode.html.md.erb:
In docs/manual/source/install/install
Github user dszeto commented on a diff in the pull request:
https://github.com/apache/predictionio/pull/471#discussion_r219937276
--- Diff: build.sbt ---
@@ -17,24 +17,7 @@
import PIOBuild._
lazy val scalaSparkDepsVersion = Map(
--- End diff --
Maybe we
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/471
Merging. Thank you!
---
Github user dszeto commented on a diff in the pull request:
https://github.com/apache/predictionio/pull/471#discussion_r219938608
--- Diff: docs/manual/source/install/index.html.md.erb ---
@@ -24,8 +24,8 @@ limitations under the License.
It is **very important** to meet
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/470
+1 from me
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/473
Thanks! Merging.
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/450
LGTM. Thank you!
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/477
LGTM. Merging. Thanks @saurabh3091 !
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/467
LGTM. Regarding automation, I think we can create scripts for some steps
inside the doc.
---
Github user dszeto commented on the issue:
https://github.com/apache/predictionio/pull/467
@shimamoto If you don't mind, please add the missing step that makes a
release on JIRA
(https://issues.apache.org/jira/projects/PIO?selectedItem=com.atlassian.jira.jira-projects-plugin
Github user dszeto commented on a diff in the pull request:
https://github.com/apache/predictionio/pull/462#discussion_r218631217
--- Diff: docker/pio/Dockerfile ---
@@ -0,0 +1,71 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
17 matches
Mail list logo
