Skip to content

Conversation

@K-cloud25
Copy link

Files Created : AutoComplete.js
AutoComplete Component.
Based on current input check with list fo branches passed as props from parent. Based on closest match suggestion is shown in terminal. useInput hook utilized to auto update value.

Updated Files : DeleteBranch.js
Replaced text Component with AutoComplete Component

Files Created : AutoComplete.js
AutoComplete Component.
Based on current input check with list fo branches passed as props from parent. Based on closest match suggestion is shown in terminal. useInput hook utilized to auto update value.

Updated Files : DeleteBranch.js
Replaced text Component with AutoComplete Component
Copy link
Contributor

@H3RSKO H3RSKO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Posted some comments, but the core functionality looks good! Maybe you can also add the AutoComplete functionality to the Checkout Branch as well?

K-cloud25 and others added 4 commits September 27, 2025 12:06
Co-authored-by: Yosef Herskovitz <34112131+H3RSKO@users.noreply.github.com>
Co-authored-by: Yosef Herskovitz <34112131+H3RSKO@users.noreply.github.com>
Co-authored-by: Yosef Herskovitz <34112131+H3RSKO@users.noreply.github.com>
Removed single quotes (') from the format for git call for branches. Ensures user does not have to type out the single quote.

Seperated the list of branchs with '|' to make the list more readable.
@K-cloud25
Copy link
Author

Hi @H3RSKO Added the requested comments to code

Added some updates to the commit as well. Please review them and let me know if any changes are required.

@K-cloud25
Copy link
Author

On a side not I would like to improve the functionality of the AutoComplete component.

If I could generate a list of suggestions instead of a singular option and one the press of the up or down key iterate through the generated list. The list could be sorted using levenshtein distance compared with the user input.

Let me know if I should work on this. I would like to work on it.

@K-cloud25 K-cloud25 requested a review from H3RSKO September 28, 2025 11:23
@K-cloud25
Copy link
Author

Hi @H3RSKO could you merge the request if all is correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants