Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle parent directory as destination #69

Merged
merged 1 commit into from
Apr 27, 2019

Conversation

jacobtolar
Copy link
Contributor

I'm open to the suggestion that I'm just doing this wrong :), but:

The project I'm working with has multiple submodules. One of these modules (two subdirectories deep) we would like to publish with this tool to the gh-pages branch, but to the top-level directory on the gh-pages branch.

E.g.:

ember github-pages:commit --message "Update gh-pages release" --destination ../../

This fails in two places:

  • rm -rf ../.. is problematic;
  • mkdir ../.. complains that it's creating a directory that already exists

To fix this I've added a regex that allows this case to be handled by the first condition of the branch.

I'm open to the suggestion that I'm just doing this wrong :), but: 

The project I'm working with has multiple submodules. One of these modules (two subdirectories deep) we would like to publish with this tool to the gh-pages branch, but to the top-level directory on the gh-pages branch.

E.g.: 
```
ember github-pages:commit --message "Update gh-pages release" --destination ../../
```

This fails in two places: 
* rm -rf ../.. is problematic; 
* mkdir ../.. complains that it's creating a directory that already exists

To fix this I've added a regex that allows this case to be handled by the first condition of the branch.
@jacobtolar
Copy link
Contributor Author

jacobtolar commented Jun 13, 2019

Hi @knownasilya, is it possible to publish the changes here to npm?

@knownasilya
Copy link
Contributor

@jacobtolar on it

@knownasilya
Copy link
Contributor

Published as v0.2.2 with dependency updates

@jacobtolar
Copy link
Contributor Author

thanks!

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