Conversation
socksy
requested review from
Malwine,
Realtin,
daveliepmann,
lislis,
plexus and
vsmart
Member
Author
|
Looks like there's some conflicts from what @vsmart editted and what the big changes we made last time. I'll try to resolve these at some point and regenerate the HTML file |
Contributor
|
I can review after the merge conflict is resolved. :)
Give the people what they want when they want, and they wants it all the time. |
Realtin
reviewed
Mar 5, 2018
| <p>NightCode is pretty easy to install, as it's just an executable JAR. We'll have a couple thumb drives to go around.</p> | ||
|
|
||
| <p>Please actually create and run a Quil project, so all dependencies are downloaded and available.</p> | ||
| <p>Please actually create and run a Quil project, so all dependencies are downloaded and available – this has caused problems before!</p> |
Member
There was a problem hiding this comment.
I thought we will only be using maria for the next workshop. Then this step is not necessary anymore
Contributor
There was a problem hiding this comment.
My plan, if we decide to have students work with maria.cloud, would be to make a separate PR to change all the materials discussing Nightcode.
Realtin
approved these changes
Mar 5, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I noticed I never made a PR for the changes we did to the coaches training last time.
Also, I had some .nix files in order to be able to compile the deck locally. Guess those can be removed if people want cleanliness.