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

feat(cmd): allow reading input from stdin #86

Merged
merged 1 commit into from
Apr 22, 2018

Conversation

raghur
Copy link
Contributor

@raghur raghur commented Apr 20, 2018

Allow reading input for stdin - libasciidoc -s -

-s, --source string the path to the asciidoc source to process. Use '-' for reading from stdin

@codecov
Copy link

codecov bot commented Apr 20, 2018

Codecov Report

Merging #86 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
+ Coverage   81.67%   81.71%   +0.03%     
==========================================
  Files          35       35              
  Lines        2096     2100       +4     
==========================================
+ Hits         1712     1716       +4     
  Misses        258      258              
  Partials      126      126

@xcoulon
Copy link
Member

xcoulon commented Apr 21, 2018

thanks for this PR, @raghur! Would you mind adding a test that reads from stdin and produces an output, please ?

@xcoulon xcoulon closed this Apr 21, 2018
@xcoulon xcoulon reopened this Apr 21, 2018
@xcoulon
Copy link
Member

xcoulon commented Apr 21, 2018

oops, closed by error, sorry about that :/

@raghur
Copy link
Contributor Author

raghur commented Apr 21, 2018

There a test for that.. Check the diff.

@@ -49,6 +54,36 @@ var _ = Describe("root cmd", func() {
GinkgoT().Logf("command output: %v", buf.String())
require.Error(GinkgoT(), err)
})

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File is not gofmt-ed with -s

@xcoulon
Copy link
Member

xcoulon commented Apr 22, 2018

ah, my bad, I missed it (facepalm)

@xcoulon xcoulon merged commit add3287 into bytesparadise:master Apr 22, 2018
@xcoulon
Copy link
Member

xcoulon commented Apr 22, 2018

thanks @raghur

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.

3 participants