no-response icon indicating copy to clipboard operation
no-response copied to clipboard

Published 20 hours ago • verified publisher icon lee-dohm

Update dependencies; adapt to latest versions

Open bugwelle opened this issue 1 year ago • 3 comments

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.

bugwelle avatar Jan 25 '24 07:01 bugwelle

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

jeffpaul avatar Feb 29 '24 17:02 jeffpaul

@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.

bugwelle avatar Mar 01 '24 09:03 bugwelle

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

jeffpaul avatar Mar 01 '24 21:03 jeffpaul