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
use interface for interactive parsers + implement in ValidatorTool #2822
Conversation
this allows us to automatically detect them from the subcommands, and makes a separate map unnecessary
package de.maibornwolff.codecharta.tools.interactiveparser | ||
|
||
interface InteractiveParser { | ||
fun callInteractive() |
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.
The name callInteractive
can create a bit of confusion. It can seem as if it's an alternative to the method call
that's needed for the CLI which includes the logic of the parsers. This method however has nothing to do with the logic. I suggest mainWithInteractiveDialog
or executeWithInteractiveDialog
Can't merge because test coverage is under 80% |
Please add a changelog entry |
[CodeCharta Analysis] Kudos, SonarCloud Quality Gate passed! |
[CodeCharta Visualization] Kudos, SonarCloud Quality Gate passed! |
use interface for interactive parsers + implement in ValidatorTool
Issue: #2341
Description
Screenshots or gifs