Michael Heap
Michael Heap
I can take a look at this on Thursday
@pmalek Please let me know when you can successfully run through this and I'll review and merge
I agree with @isaacsanders. I'd be happy to merge a `GH_HOST` PR
This isn't as easy as it appears sadly 😢 Octokit has moved to ESM modules away from CommonJS, which means some fairly involved work to update the rest of the...
ESM is no longer an issue, but we make heavy use of `nock` in the tests to check behaviour. Octokit 18->20 switches to `fetch` via `undici`, which is not yet...
The sample workflow I shared uses `author_association`. You might be more interested in checking if they have `write` permission. See https://michaelheap.com/github-actions-check-permission/ for more info
The label approach works, but you _do_ lose some metadata as it's technically a different event triggering the job. If you're comfortable with the "maintainer re-runs the job" it's the...
@gloaysa Could you please make sure tests pass locally?