Conversation
|
commit: |
sacrosanctic
left a comment
There was a problem hiding this comment.
first time using the review feature, i think i figure it out now.
|
|
||
| ```sh | ||
| npx sv add [PROTOCOL][COMMUNITY_ADDON] | ||
| npx sv add [addon_package_name] |
There was a problem hiding this comment.
need clarification, the uses will see @my-org/sv, but they need to type @my-org?
There was a problem hiding this comment.
This is the general statement
PROTOCOL: @ or file:
| # Scoped package: @org/sv | ||
| npx sv add @supacool | ||
|
|
||
| # Regular npm package (with or without scope) |
There was a problem hiding this comment.
what does with or without scope mean?
There was a problem hiding this comment.
In npm:
The scope name is everything between the @ and the slash
|
|
||
| ```sh | ||
| npx sv add eslint "@supacool" | ||
| npx sv add eslint @supacool |
There was a problem hiding this comment.
are the quotes required?
There was a problem hiding this comment.
On windows yes :/
On the rest no
I agree that it's way better wo the quote!
Maybe we could add "On windows do like this..." ? (But it's also kinda ungly)
| ``` | ||
|
|
||
| ## Add-on structure | ||
| The project has a `README.md` and `CONTRIBUTING.md` to guide you along. |
There was a problem hiding this comment.
wheres the source?
There was a problem hiding this comment.
You think we should remove the chapter?
There was a problem hiding this comment.
The diff is inaccurate, the section is just renamed. I'm asking about the readme.md and contributing.md.
|
|
||
| > [!NOTE] | ||
| > Add the `sv-add` keyword so users can discover your add-on on npm. | ||
|
|
||
| ### Export options | ||
|
|
||
| Your package can export the add-on in two ways: |
There was a problem hiding this comment.
this section is confusing. what are the package names? will sv look for with /sv and without?
There was a problem hiding this comment.
Hummm, not sure to understand your question. Maybe because it's not linked with the right part of the code?!
Yes, we are checking root export and /sv export
Closes nothing
Description
Checklist