search

HiDeoo / starlight-package-managers

Quickly display npm related commands for multiple package managers in your Starlight documentation site
https://starlight-package-managers.vercel.app
MIT License
22 stars 3 forks source link

feat: integrate pkg.pr.new #12

Closed o-az closed 2 months ago

o-az commented 2 months ago

integrates https://github.com/stackblitz-labs/pkg.pr.new Why

Why are these changes necessary? so that we have an option to very easily test changes while in pr (before merge)

How

How were these changes implemented? github action

Screenshots

If applicable, add screenshots to help explain what is being modified.

I also updated action versions to current.

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
starlight-package-managers ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 2, 2024 4:20am
HiDeoo commented 2 months ago

Thanks for your contribution.

I don't think this small component is at a level where using something like pkg.pr.new is necessary and the current documentation acting as a playground combined to the tests included in the project should be enough to test PR changes. The documentation/playground can also temporarily include extra demo/test pages during the early stages of a PR which can be removed later.

I would highly recommend to open issues for any feature requests or changes before starting to work on them to avoid unnecessary work. Thanks for the contribution nonetheless.