-
Notifications
You must be signed in to change notification settings - Fork 172
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
Move CLI to its own subproject #246
Labels
Milestone
Comments
5 tasks
abelsromero
added a commit
to abelsromero/asciidoctorj
that referenced
this issue
Mar 21, 2023
* Create new sub-module asciidoctorj-cli: migrated 'org.asciidoctor.jruby.cli' classes and related tests to it. * Created CliOptions in AsciidoctorUtils from values in AsciidoctorCliOptions to avoid circular dependency. * Updated README with new sub-module information. * Fix jar names in command in README. * Minor refactors and fixes. Fixes asciidoctor#246
abelsromero
added a commit
to abelsromero/asciidoctorj
that referenced
this issue
Mar 21, 2023
* Create new sub-module asciidoctorj-cli: migrated 'org.asciidoctor.jruby.cli' classes and related tests to it. * Created CliOptions in AsciidoctorUtils from values in AsciidoctorCliOptions to avoid circular dependency. * Updated README with new sub-module information. * Fix jar names in command in README. * Minor refactors and fixes. Fixes asciidoctor#246
abelsromero
added a commit
to abelsromero/asciidoctorj
that referenced
this issue
Mar 21, 2023
* Create new sub-module asciidoctorj-cli: migrated 'org.asciidoctor.jruby.cli' classes and related tests to it. * Created CliOptions in AsciidoctorUtils from values in AsciidoctorCliOptions to avoid circular dependency. * Updated README with new sub-module information. * Fix jar names in command in README. * Minor refactors and fixes. Fixes asciidoctor#246
abelsromero
added a commit
to abelsromero/asciidoctorj
that referenced
this issue
Mar 23, 2023
* Create new sub-module asciidoctorj-cli: migrated 'org.asciidoctor.jruby.cli' classes and related tests to it. * Created CliOptions in AsciidoctorUtils from values in AsciidoctorCliOptions to avoid circular dependency. * Updated README with new sub-module information. * Fix jar names in command in README. * Minor refactors and fixes. Fixes asciidoctor#246
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I think we should move the CLI to its own subproject named asciidoctorj-cli. That way, we don't mix API concerns with CLI concerns. Now that we have a distribution, there is no additional overhead for the user in having a separate module since the distribution worries about setting up the classpath appropriately.
The text was updated successfully, but these errors were encountered: