sveltejs / cli

The Svelte CLI
MIT License
258 stars 11 forks source link

feat: short description for each add-on #299

Open benmccann opened 1 week ago

benmccann commented 1 week ago

closes https://github.com/sveltejs/cli/issues/297 closes https://github.com/sveltejs/cli/issues/247

I've seen this feedback a number of times now, so think it's worth doing something here if we can keep the design clean enough

Screenshot from 2024-11-13 09-39-37

changeset-bot[bot] commented 1 week ago

🦋 Changeset detected

Latest commit: 322110d7e306614b5117acb6e86f38282cc3593c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ---- | ----- | | sv | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

pkg-pr-new[bot] commented 1 week ago

Open in Stackblitz

pnpm add https://pkg.pr.new/sveltejs/cli/sv@299
pnpm add https://pkg.pr.new/sveltejs/cli/svelte-migrate@299

commit: 322110d

AdrianGonz97 commented 1 week ago

I don't know, #297 is the first time that I've seen a complaint relating to this. I'm also having a hard time finding any similar complaints when I search through the discord and reddit. Not sure if this is a real issue that people actually face

manuel3108 commented 1 week ago

Agree with @AdrianGonz97, haven't seen any feedback beside the one mentioned earlier

benmccann commented 1 week ago

I don't know if I'm going to be able to go back and find them, but there is https://github.com/sveltejs/cli/issues/253:

For some addons it's pretty obvious what they do, but for others a link or a description would be very helpful. For example, unless you already know what mdsvex is, you're unlikely to know that that's what you want to be able to use Markdown in your Svelte components.

That issue was referring to our docs, but it's just as true that we don't have a description in the CLI

And of course there have been lots of folks confused about the Lucia adder (e.g. https://github.com/sveltejs/cli/issues/217, https://github.com/sveltejs/cli/issues/189, on reddit, etc.). This would allow us to close https://github.com/sveltejs/cli/issues/247 as well, by clarifying that it's the guide in the hint.

manuel3108 commented 1 week ago

Already forgot about the the lucia stuff, convinced.

manuel3108 commented 6 days ago

Rebasing basically resulted in rewrite. I hate merge conflicts!