cmds/core/gosh: replace go-prompt with bubbline#2606
Conversation
229f006 to
f306a5d
Compare
|
TODO: Fix tests |
|
we probably ought to get this done or close it. |
I'll be getting it in shape this week and fix the unit tests. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2606 +/- ##
==========================================
+ Coverage 69.07% 72.13% +3.05%
==========================================
Files 112 354 +242
Lines 6452 37386 +30934
==========================================
+ Hits 4457 26970 +22513
- Misses 1995 10416 +8421
... and 249 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Ready for review and testing |
As go-prompt used to cause issues this commit is replacing it. The bubbline library is actively maintained and more features complete than go-prompt was. This also addresses ctrl-c handling, a proper history and better autocomplete. Signed-off-by: Marvin Drees <marvin.drees@9elements.com>
Signed-off-by: Marvin Drees <marvin.drees@9elements.com>
Signed-off-by: Marvin Drees <marvin.drees@9elements.com>
Signed-off-by: Marvin Drees <marvin.drees@9elements.com>
Signed-off-by: Marvin Drees <marvin.drees@9elements.com>
|
Did we agree it is a repo we can rely on ? -- asking to see if it is worth importing in for our internal use case. |
|
ping |
|
Well go-prompt is dead and has several issue that need fixing. I don't have the capacities to do so, so if you want to maintain go-prompt and fix all the open issues then we can revert that change. Bubbline is a fairly young project, that's why there is not so much "public interest" yet (also not sure how you measure that). Bubbline has none of the aforementioned issues and Raphael is still actively maintaining this repository. I'm not familiar with your internal guidelines but from a purely open-source perspective bubbline fixes more issues than it creates. |
|
One of the the main issues raised here is that Bubbleline itself assembles features implementations from an array of other new GitHub repos. Pulling in new uroot, means importing in Bubbleline into our software supply chain. That also mean we import / download the same array of GitHub repos bubbline depends on to be able to build bubbline. Bubbline repo is not self sufficient (on standard golang libraries). We are looking at 2+ weeks of SWE hours to set up all dependencies needed by bubbline. I don't believe we need the new features, but we would need bear the setup cost that bubbline brings in. |
|
ping |
As go-prompt used to cause issues this commit is replacing it. The bubbline library is actively maintained and more features complete than go-prompt was. This also addresses ctrl-c handling, a proper history and better autocomplete.