Add clarification of working directory to start of Tracking Changes section #330

Open
wants to merge 1 commit into
from

Projects

None yet

3 participants

@rgaiacs
Contributor
rgaiacs commented Aug 27, 2016

Cover #283 and #285.

@brainwane
brainwane commented Oct 29, 2016 edited

I'm wondering whether we should add sample expected output, e.g. /home/user/planets with the {: .output} formatting. But, since every learner will have a different path, that might cause more confusion than it addresses.

@rgaiacs
Contributor
rgaiacs commented Oct 31, 2016

I'm wondering whether we should add sample expected output, e.g. /home/user/planets with the {: .output} formatting. But, since every learner will have a different path, that might cause more confusion than it addresses.

@brainwane I didn't understand how your comment is related with my pull request.

@brainwane

Hi @rgaiacs -- sorry for not being clear. I was suggesting that, since your pull request includes a suggested addition where the instructor demonstrates an input (invoking pwd on the command line), it might be a good idea to also include an addition of sample output. But I'm also not absolutely certain of this and willing to be told "bad idea" or "let's put that in a separate issue". :)

@daisieh
Contributor
daisieh commented Oct 31, 2016

I think this looks good as-is. Usually the instructor is live-coding this as it happens, so the class would see the output.

@brainwane

OK, thanks @daisieh! Suggestion withdrawn. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment