microsoft / inshellisense

IDE style command line auto complete
MIT License
8.21k stars 179 forks source link

fix: reset the selected completion index to 0 to prevent the selected… #173

Closed alanhe421 closed 4 months ago

alanhe421 commented 4 months ago

I found that the default selected index for completion needs to be reset to avoid the default selected index still being the previously selected one when the completion is displayed.

Please help to review this. @cpendery

cpendery commented 4 months ago

@alanhg, thanks for the fix! If you could fix the formatting by running the lint:fix command before committing that would be awesome. Pushing the lint fixes to your PR now

alanhe421 commented 4 months ago

@cpendery

I am using WebStorm, and it seems that the eslint configuration that was not applied to successfully walk the project.

Here in order to avoid such problems, I will submit another husky to ensure that the submit automatically executes lint to avoid such things.

Thanks.