-
Notifications
You must be signed in to change notification settings - Fork 173
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
Modernize cli #1067
Modernize cli #1067
Conversation
9e3f8ee
to
0aae689
Compare
+ "' missing or cannot be read"); | ||
}); | ||
|
||
return inputFiles.stream() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I refactored this as-is, but the output is actually never used, maybe we should remove and just cal convert "for each" input?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that would make sense. The method is also not part of any public API, so we can just remove the return value.
This looks good to me. |
Yes, thanks. I can find some time today. |
964d375
to
7e0d8de
Compare
* Removed unused imports * Applied Java8+ constructs * Formatting: unified code to use spaces * Remove AsciidoctorJ deprecated APIs when possible * Use AsciidoctorCliOptions.TIMINGS_OPTION_NAME constant in AsciidoctorInvoker
7e0d8de
to
6ae7609
Compare
Done 🥳 |
Awesome, thank you! 🙏🎉 |
Thank you for opening a pull request and contributing to AsciidoctorJ!
Please take a bit of time giving some details about your pull request:
Kind of change
Description
What is the goal of this pull request?
How does it achieve that?
Are there any alternative ways to implement this?
Are there any implications of this pull request? Anything a user must know?
Test pass and I have not gone far in code changes just in case.
Issue
If this PR fixes an open issue, please add a line of the form:
Fixes #Issue
Release notes
Please add a corresponding entry to the file CHANGELOG.adoc