clue ยท GitHub

Conversation

@clue

@clue clue commented

Dec 10, 2017

โ€ข

edited

Loading

Copy link Copy Markdown

Member

We now support middleware request handlers by default and no longer require an explicit call to MiddlewareRunner. Accordingly, the MiddlewareRunner is now internal only.

Builds on top of #215

@WyriHaximus

WyriHaximus

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes perfect sense to me ๐Ÿ‘

jsor

jsor approved these changes Dec 11, 2017

@jsor

@jsor jsor mentioned this pull request

Dec 11, 2017

Merged

@jsor

Copy link Copy Markdown

Member

Test failing after merging to master. Not sure why now and not in the PR build. Opened #279 with a fix.

@clue

clue deleted the middleware branch

December 11, 2017 07:46

Merged

@clue clue mentioned this pull request

Dec 12, 2017

Merged

Read the original on github.com โ†—