Eli Skeggs

Results 119 comments of Eli Skeggs

> I think there is no more need for this PR. GitHub App authentication can be implemented so: > > ``` > integration = GithubIntegration(app_id, private_key) > installation_id = integration.get_installation(org,...

I'm currently working around this in a (hopefully) forward-compatible way like: ```js import * as commitlint from '@commitlint/core'; import interopRequireDefault from '@babel/runtime/helpers/interopRequireDefault'; const format = interopRequireDefault(commitlint.format), lint = interopRequireDefault(commitlint.lint), load...

I could rebase, sure. That said, I would favor #186 over this pull request since you seem to be comfortable making breaking changes to `bee-queue`. Was the decision to close...

Ah, fair enough then. I don't recall these discussions or this PR well enough to green-light anything, but I'll try and find some time to rebase over the next few...

> Warning diagnostics, which are visible in the user interface, however they do not prevent further Terraform execution. My primary use-case is for `external` data sources in either root workspaces...

Rebased, largely to reconcile against the changes in #134. Unfortunately, it seems the tests are flaky. I don't know if this is a new upstream regression, or something that changed...

> Could you describe your setup? I'm getting intermittent and inconsistent test failures when I run the tests locally on my M1 MacBook Pro with Redis 6.2 and Node 16.17.1....

I'm running into the same issue. The request body contains the id/secret, when some services require it to be in the Authorization header as a Basic auth request. I'm not...

FYI, we no longer implement `checkDNS` in `isemail`. That use-case will be filled via the `parse` method, whereby users of the library can run their own DNS query against the...

I have no idea what that would entail - is it just the addition of a types file? By open dev branch do you mean somehow modifying branch-level permissions? I'd...