lee-dohm / no-response

A GitHub Action that closes Issues where the author hasn't responded to a request for more information
MIT License
38 stars 14 forks source link

Update dependencies; adapt to latest versions #410

Open bugwelle opened 8 months ago

bugwelle commented 8 months ago

This commit updates this action to the latest Octokit version, as well as updates all other dependencies to their latest available versions.

There were a few changes necessary to make this work: Octokit now uses a .rest property. I've removed scramjet as its functionality can be replaced by async+ Promise.all in this library.

Furthermore, to allow developers to dispatch this action manually, I've added the workflow_dispatch event. The event type will be logged.

jeffpaul commented 7 months ago

@bugwelle mind porting this update to https://github.com/MBilalShafi/no-response-add-label as a forked version of this repo/action?

bugwelle commented 7 months ago

@jeffpaul That repository has diverged from this original repository. It also partially uses the new octokit API, it seems:

https://github.com/lee-dohm/no-response/compare/main...MBilalShafi:no-response-add-label:main#diff-63a9c84fe8adae96e8e39100541421adf38bf5e2170c2280923554526ca33213R137

Porting the changes doesn't seem trivial (or necessary).

The fork-author also changed the MIT license without keeping @lee-dohm in it, which I find… let's say I'm not a fan of it. 😄 It also has no "Issues" tab, which is why I'd avoid that fork.

jeffpaul commented 7 months ago

@bugwelle yeah I opened a PR to correct the MIT licensing bit: https://github.com/MBilalShafi/no-response-add-label/pull/6.