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

Add and document bin/md-readme #5

Merged
merged 2 commits into from May 6, 2015
Merged

Conversation

@NateEag
Copy link
Contributor

@NateEag NateEag commented May 5, 2015

The intention is to make life easier for Cask users depending on
md-readme.el in development environments.

It could be useful in other contexts, though.

The intention is to make life easier for Cask users depending on
md-readme.el in development environments.

It could be useful in other contexts, though.
@NateEag
Copy link
Contributor Author

@NateEag NateEag commented May 5, 2015

Resolves issue #4.

@thomas11

This comment has been minimized.

Copy link

@thomas11 thomas11 commented on bin/md-readme in 9e07815 May 5, 2015

[[ is Bash's extended test which is not in sh. So the shebang line should either be changed to /bin/bash, or the test should be rewritten to use sh's test.
http://tldp.org/LDP/abs/html/testconstructs.html#DBLBRACKETS

This comment has been minimized.

Copy link
Owner Author

@NateEag NateEag replied May 5, 2015

dang - I actually meant to fix that but obviously failed to do so.

Just pushed a commit to fix it.

thomas11 added a commit that referenced this pull request May 6, 2015
Add and document bin/md-readme
@thomas11 thomas11 merged commit 9f3630d into thomas11:master May 6, 2015
@thomas11
Copy link
Owner

@thomas11 thomas11 commented May 6, 2015

Merged, thanks Nate!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants