Skip to content
This repository was archived by the owner on Mar 11, 2026. It is now read-only.

fix(middleware): tweak the middleware api#291

Merged
ofrobots merged 3 commits intogoogleapis:masterfrom
ofrobots:tweak-middleware-api
Oct 31, 2018
Merged

fix(middleware): tweak the middleware api#291
ofrobots merged 3 commits intogoogleapis:masterfrom
ofrobots:tweak-middleware-api

Conversation

@ofrobots
Copy link
Copy Markdown
Contributor

  • Child suffix is not necessary.
  • Make emitRequestLog optional. Make it control finish tracking.

* Child suffix is not necessary.
* Make emitRequestLog optional. Make it control finish tracking.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 30, 2018
@kjin kjin self-requested a review October 30, 2018 22:38
Comment thread src/middleware/express/make-middleware.ts Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 31, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@2c1c8dc). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #291   +/-   ##
=========================================
  Coverage          ?   92.31%           
=========================================
  Files             ?       14           
  Lines             ?      638           
  Branches          ?       61           
=========================================
  Hits              ?      589           
  Misses            ?       35           
  Partials          ?       14
Impacted Files Coverage Δ
src/middleware/express/make-middleware.ts 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c1c8dc...4d557d5. Read the comment docs.

@ofrobots ofrobots merged commit c41db2f into googleapis:master Oct 31, 2018
@ofrobots ofrobots deleted the tweak-middleware-api branch October 31, 2018 03:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants