-
Notifications
You must be signed in to change notification settings - Fork 2
feat: Add Args documentation #12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks for the PR!
I think we could delete the TBD pages for now to merge the new section in, and then split the new content to separate PRs. Will probably do something similar with Tab as well 🫡
@@ -11,6 +11,7 @@ | |||
}, | |||
"dependencies": { | |||
"@astrojs/starlight": "^0.32.2", | |||
"@bomb.sh/args": "^0.3.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm do we need to install this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have to check, but I'm pretty sure that without it twoslash won't be able to do its magic.
But maybe it can be moved into dev dependencies ? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I confirm, without it twoslash don't work.
But it can be moved to devDependencies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, it is required when working with twoslash.
label: "Args", | ||
id: "args", | ||
icon: "seti:shell", | ||
link: "/args/getting-started", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙌
Thank you both 🫡 |
TODO
Will be done in a later PR