Option selector in Prompt refactor

BuilderIO / ai-shell

A CLI that converts natural language to shell commands.

MIT License

4.22k stars 278 forks source link

Option selector in Prompt refactor #63

Closed Rafaelcv7 closed 1 year ago

Rafaelcv7 commented 1 year ago

Refactor mentioned in Issue #61

@steve8708 tried to set the configuration for unit testing with Jest, but found out that there is a problem when trying to import prompt.ts to jest test for mocking. :/ It seems to be about the execaCommand import in prompt.ts

But I did exhaustive manual testing on this refactor hope that is worth it. Let me know if anything else.

steve8708 commented 1 year ago

Looks good to me! Just one minor request

Rafaelcv7 commented 1 year ago

Looks good to me! Just one minor request

Done! I may missed that part XD, thanks for the correction!