feat: `createNodeMiddleware()`#509

Merged

gr2m merged 21 commits intomasterfrom

507/createNodeMiddleware

Mar 30, 2021

Commits on Mar 29, 2021

refactor: move current `createMiddleware` implementation to `src/middleware-legacy`
gr2m
committed
docs(README): `createNodeMiddleware()`. Remove obsolete documentation for `new Webhooks({ path })` and `webhooks.middleware`
gr2m
committed
WIP
gr2m
committed
adapt for #514
gr2m
committed
test: make `createNodeMiddleware()` tests pass
gr2m
committed
refactor: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794205
gr2m
committed
refactor: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794954
gr2m
committed
refactor: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602795056
gr2m
committed
test: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794699
gr2m
committed
test: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794566
gr2m
committed
make it work with node 10 & 12
gr2m
committed
build(dev-deps): express
gr2m
committed
build(package): lock file
gr2m
committed
correctly handle usage with express. No need to check for correct route, express is taking care of routing
gr2m
committed
test: 100% coverage
gr2m
committed
TypeScript: use types of `Console` for log methods
gr2m
committed
feat: `createNodeMiddleware(express, { log })` option
gr2m
committed
remove @types/express to workaround stupid TypeScript errors
gr2m
committed
test: workaround requiring types for express
gr2m
committed
feat: deprecate `path` option and `webhooks.middleware`
gr2m
committed
docs(README): remove `webhooks.middleware` from API toc
gr2m
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: `createNodeMiddleware()`#509

feat: `createNodeMiddleware()`#509
gr2m merged 21 commits intomasterfrom
507/createNodeMiddleware

Commits on Mar 29, 2021

refactor: move current `createMiddleware` implementation to `src/middleware-legacy`

docs(README): `createNodeMiddleware()`. Remove obsolete documentation for `new Webhooks({ path })` and `webhooks.middleware`

WIP

adapt for #514

test: make `createNodeMiddleware()` tests pass

refactor: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794205

refactor: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794954

refactor: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602795056

test: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794699

test: address https://github.com/octokit/webhooks.js/pull/509\#discussion_r602794566

make it work with node 10 & 12

build(dev-deps): express

build(package): lock file

correctly handle usage with express. No need to check for correct route, express is taking care of routing

test: 100% coverage

TypeScript: use types of `Console` for log methods

feat: `createNodeMiddleware(express, { log })` option

remove @types/express to workaround stupid TypeScript errors

test: workaround requiring types for express

feat: deprecate `path` option and `webhooks.middleware`

docs(README): remove `webhooks.middleware` from API toc

feat: `createNodeMiddleware()`#509gr2m merged 21 commits intomasteroctokit/webhooks.js:masterfrom 507/createNodeMiddlewareoctokit/webhooks.js:507/createNodeMiddlewareCopy head branch name to clipboard

Commits

Commits on Mar 29, 2021

feat: `createNodeMiddleware()`#509
gr2m merged 21 commits intomasterfrom
507/createNodeMiddleware