nicoalbanese / kirimase

Build full-stack Next.js apps, incredibly fast

https://kirimase.dev

MIT License

2.39k stars 107 forks source link

Remove @inquirer/checkbox #131

Closed Kairedblue closed 3 months ago

Kairedblue commented 4 months ago

Description

Remove invalid import { Choice } from "@inquirer/checkbox"

Create a new Choice type

Result

Choice type

type Choice<Value> = {
name?: string;
value: Value;
disabled?: boolean | string;
checked?: boolean;
type?: never;
};

afogel commented 4 months ago

I was wondering about this, too. In my commits, I basically did the same thing, but didn't commit it since it seemed out of scope for the work I was doing. Curious whether this is also throwing an error in your build @nicoalbanese?

nicoalbanese commented 3 months ago

I was wondering about this, too. In my commits, I basically did the same thing, but didn't commit it since it seemed out of scope for the work I was doing. Curious whether this is also throwing an error in your build @nicoalbanese?

Hey! I haven't ran into any build issues with this but I'm happy to merge this as it removes a dependency