Skip to content

slight tidy and example#360

Merged
QuincyLarson merged 1 commit intofreeCodeCamp:masterfrom
benmcmahon100:patch-1
Apr 25, 2015
Merged

slight tidy and example#360
QuincyLarson merged 1 commit intofreeCodeCamp:masterfrom
benmcmahon100:patch-1

Conversation

@benmcmahon100
Copy link
Copy Markdown
Member

Replaced one occurrence of gitter with slack and added an example bit torrent client after someone pointed out in the help section that there was not guide to demonstrate how to do so.

Replaced one occurrence of gitter with slack and added an example bit torrent client after someone pointed out in the help section that there was not guide to demonstrate how to do so.
@terakilobyte
Copy link
Copy Markdown
Contributor

We will look at this today. I need to discuss with Quincy whether we are still pushing/supporting the vm since we are moving everyone towards c9.io.

@terakilobyte terakilobyte added status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. and removed todo labels Apr 24, 2015

This comment was marked as off-topic.

@QuincyLarson
Copy link
Copy Markdown
Contributor

We found a syntax error in the JSON (be sure to actually run the code before you submit a pull request, as this would have resulted in a fatal error). Also, we have decided to move away from using virual machines in favor of developing on c9.io (where several of our campers are building their nonprofit projects). We will remove this copy completely. Thanks for catching the mention of gitter, though. We'll be sure to fix that. Thanks again for your pull request.

QuincyLarson pushed a commit that referenced this pull request Apr 25, 2015
@QuincyLarson QuincyLarson merged commit 252f955 into freeCodeCamp:master Apr 25, 2015
@QuincyLarson QuincyLarson removed the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Apr 25, 2015
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.

4 participants