Skip to content

docs: Add development#161

Merged
gaocegege merged 3 commits intomasterfrom
doc/install
Jul 15, 2017
Merged

docs: Add development#161
gaocegege merged 3 commits intomasterfrom
doc/install

Conversation

@gaocegege
Copy link
Copy Markdown
Member

@gaocegege gaocegege commented Jul 14, 2017

I think this PR needs your review @jeremydouglass

Changes Preview

Close #159 🙏

Signed-off-by: Ce Gao ce.gao@outlook.com

Signed-off-by: Ce Gao <ce.gao@outlook.com>
@gaocegege gaocegege requested a review from jeremydouglass July 14, 2017 02:27
Signed-off-by: Ce Gao <ce.gao@outlook.com>
@jeremydouglass
Copy link
Copy Markdown
Member

jeremydouglass commented Jul 14, 2017

While reviewing this and making minor corrections and improvements: I am not sure why howto.md should be separate from readme.md.

Is there a reason not to just merge the contents of howto.md into readme.md and delete howto.md?

@jeremydouglass
Copy link
Copy Markdown
Member

Is the docker image being updated? I glanced at quay.io/gaocegege/processing.r and it showed auto-builds from master that are only a few hours old. I didn't try it.

@gaocegege
Copy link
Copy Markdown
Member Author

LGTM, quay.io/gaocegege/processing.r is just like Travis, it builds the new docker image automatically, but we still can not run 3D sketches in the docker container. It is deprecated but the source code in docker is the most updated.

@gaocegege gaocegege merged commit 0a5eee3 into master Jul 15, 2017
@gaocegege gaocegege deleted the doc/install branch July 15, 2017 02:15
@gaocegege gaocegege modified the milestone: Evaluation 2 Jul 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants