Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Fixed install-with-docker link #505

Merged
merged 1 commit into from
Jan 17, 2019
Merged

Fixed install-with-docker link #505

merged 1 commit into from
Jan 17, 2019

Conversation

mmoreram
Copy link
Contributor

No description provided.

README.md Outdated
@@ -45,7 +45,7 @@ A few installation options available.
* [Installing Apache PredictionIO from
Binary/Source](http://predictionio.apache.org/install/install-sourcecode/)
* [Installing Apache PredictionIO with
Docker](http://predictionio.apache.org/community/projects/#docker-images)
Docker](http://predictionio.apache.org/install/install-docker/)
(community contributed)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this @mmoreram, I forgot to modify README while changing the docker doc.
Can you also remove the community contributed tag?
Cus the new link that you are pointing to is not community projects.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're welcome :)
Updated!
Happy to help!

@shimamoto
Copy link
Member

@mmoreram LGTM. Thanks!

@Wei-1 Thanks for review.

@shimamoto shimamoto merged commit aa79f7c into apache:develop Jan 17, 2019
@mmoreram mmoreram deleted the patch-1 branch January 17, 2019 10:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants