Feat/revert wizard#56
Conversation
- Moved wizards to a seperate package - Introduced `Wizard` interface for consistency - Refactored the key press management into `keys.go` with action constants - Extracted UI components to `internal/tui/components` - Added `commands.go` to unify gitx function calls - Moved the search functionality to a new `search` package - Moved theme to a different package to avoid circular dependencies
|
+4k/-2.5k change is rather hard to review.. also from the code I notice there are multiple unneeded features, and the build ci fails If you would please revise the code and clean it up with a shorter, cleaner PR Feel free to start a new one for your ease :) |
|
Hey @kubeden I have actually started working from PR #55 (refactored version). so most of the changes you see are are from the commits to that branch. once that is merged you can see the original changes. Please don't worry, if there is some trouble after that one is merged then I will reopen a new fresh PR. |
|
Hmm.. I understand. Let me clone and explore. I'll let you know. In the meantime, please change the target branch to |
|
@kubeden I was not aware that you had the |
|
Ignore the refactor for now! You can do the change towards the un-refactored branch and I'll refactor myself later |
3 participants
Summary
Describe the change briefly.
Linked Issue(s)
Closes #53
Checklist
go buildandgo test ./...