-
Notifications
You must be signed in to change notification settings - Fork 31
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
Tech/3483/integrate kotter #3609
base: main
Are you sure you want to change the base?
Conversation
6282905
to
11807b5
Compare
I rebased the changes on-top of the current main branch. |
Quality Gate passed for 'CodeCharta Visualization'Issues Measures |
This is blocked by flaky tests that we think are caused by a bug in kotters testing library (we created an issue for that: varabyte/kotter#114) |
Class is no longer needed as all functions extend the Session class from Kotter and are applicable in the correct scope
Still a lot of todos that need to be closed and comments that need to be implemented, as well as finding a solution for the session problem
…3483) Also added defaultCallback to reduce tests getting stuck on dropped input for debugging purposes. Swapped to inmemory versions of kotter press and type functions
efb4e7e
to
5abd1e1
Compare
Quality Gate passed for 'CodeCharta Visualization'Issues Measures |
Quality Gate failed for 'CodeCharta Analysis'Failed conditions |
Rebuild functionality of the no longer supported KotlinInquirer using Kotter
Issue: #3483
Description
Use the Kotter library to rebuild the functionality of the KolinInquirer Library. It is currently used for the interactive CLI interface but is no longer supported.
Using our own questions built with kotter gives us more control over the interactive questions and allows for features like auto completion.
This also integrates the new kotter methods into the StructureModifier.
Definition of Done
A PR is only ready for merge once all the following acceptance criteria are fulfilled:
Screenshots or gifs