Skip to content

Allow specifying demo directory#186

Merged
kytrinyx merged 2 commits intoexercism:masterfrom
Tonkpils:demo-dir
Apr 24, 2015
Merged

Allow specifying demo directory#186
kytrinyx merged 2 commits intoexercism:masterfrom
Tonkpils:demo-dir

Conversation

@Tonkpils
Copy link
Copy Markdown
Contributor

Fixes #180 for the demo command.

Paths of the form /foo/~/bar/ should not be expanded with the home directory
The directory will be represented as follows:
    - 'mydirectory' will be a relative directory in the current working dir
    - '~/mydir' wil expand be a directory in the home directory
@kytrinyx
Copy link
Copy Markdown
Member

This looks great. For a moment I was confused about the c.SetDir(c.Dir), but I realized that this lets us sanity-check and have good fallbacks. Makes sense!

kytrinyx added a commit that referenced this pull request Apr 24, 2015
Allow specifying demo directory
@kytrinyx kytrinyx merged commit 139105a into exercism:master Apr 24, 2015
@Tonkpils Tonkpils deleted the demo-dir branch April 24, 2015 21:10
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.

exercise download directory defaults to home dir

2 participants