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

fix: in Koa2 interface await next as a function#336

Merged
DominicKramer merged 2 commits intogoogleapis:masterfrom
DominicKramer:fix/koa2-next-is-a-function
Apr 9, 2019
Merged

fix: in Koa2 interface await next as a function#336
DominicKramer merged 2 commits intogoogleapis:masterfrom
DominicKramer:fix/koa2-next-is-a-function

Conversation

@DominicKramer
Copy link
Copy Markdown
Contributor

Fixes: #335

@DominicKramer DominicKramer requested a review from a team April 9, 2019 16:07
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 9, 2019
DominicKramer referenced this pull request Apr 9, 2019
This change introduces a Koa2 plugin that is available through the `koa2` property.
The existing Koa1 plugin is still accessed through the `koa` property.

Fixes: #8
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2019

Codecov Report

Merging #336 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #336   +/-   ##
======================================
  Coverage    96.1%   96.1%           
======================================
  Files          34      34           
  Lines        1694    1694           
  Branches       76      76           
======================================
  Hits         1628    1628           
  Misses         49      49           
  Partials       17      17

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 4bd26cd...f66a56e. Read the comment docs.

Copy link
Copy Markdown
Contributor

@ofrobots ofrobots left a comment

Choose a reason for hiding this comment

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

Can we also a test that exercises this code?

@DominicKramer
Copy link
Copy Markdown
Contributor Author

@ofrobots I'm in the process of creating the tests, and I think they should be samples tests. In particular, I am updating the samples to include samples for the different frameworks. Then I'll update the samples tests to test each sample.

@DominicKramer
Copy link
Copy Markdown
Contributor Author

Since the sample test changes involve a bit of work, I made an issue (#337) to track it. A follow-up PR will implement that work, and thus I will land this PR. That way, each PR can focus on a specific task.

@DominicKramer DominicKramer merged commit 5838e49 into googleapis:master Apr 9, 2019
@codyzu
Copy link
Copy Markdown

codyzu commented Apr 10, 2019

Repeating my comment on the issue, this fixed the koa interface. Can you fix the koa2 interface, as was described in the 0.6.2 release changelog (and the name of the PR 😉 )?

DominicKramer added a commit to DominicKramer/nodejs-error-reporting that referenced this pull request Apr 11, 2019
PR googleapis#336 fixed the Koa plugin, but didn't fix the Koa2 plugin.

Fixes: googleapis#335
DominicKramer added a commit that referenced this pull request Apr 11, 2019
PR #336 fixed the Koa plugin, but didn't fix the Koa2 plugin.

Fixes: #335
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.

5 participants