Don't run `pkg.pr.new` if there aren't changes to the `nuqs` package

47ng / nuqs

Type-safe search params state manager for React frameworks - Like useState, but stored in the URL query string.

https://nuqs.47ng.com

MIT License

4.91k stars 105 forks source link

Don't run `pkg.pr.new` if there aren't changes to the `nuqs` package #655

Closed franky47 closed 1 month ago

franky47 commented 2 months ago

pkg.pr.new is a service by StackBlitz to deploy previews of NPM packages without polluting the NPM registry (it's pretty cool).

At the moment the PR preview workflow (pkg.pr.new.yml) runs on any PR.

This is kind of wasteful on pkg.pr.new's servers and resources: we should abort if the PR introduces no changes to what's actually being deployed.

PRs welcome!

chbg commented 1 month ago

Hi, @franky47 ! I’ve submitted a pull request that addresses this issue by adding a conditional check to pkg.pr.new.yml to prevent unnecessary runs when there are no changes to the nuqs package. You can view the PR here. Looking forward to your feedback!

github-actions[bot] commented 1 month ago

:tada: This issue has been resolved in version 1.20.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: