Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

Python: Add routing header #1658

Merged
merged 6 commits into from
Nov 11, 2017
Merged

Conversation

evaogbe
Copy link
Contributor

@evaogbe evaogbe commented Oct 25, 2017

@codecov-io
Copy link

codecov-io commented Oct 25, 2017

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1658   +/-   ##
=========================================
  Coverage          ?   84.98%           
  Complexity        ?     4383           
=========================================
  Files             ?      425           
  Lines             ?    17452           
  Branches          ?     1822           
=========================================
  Hits              ?    14831           
  Misses            ?     2004           
  Partials          ?      617
Impacted Files Coverage Δ Complexity Δ
...api/codegen/viewmodel/OptionalArrayMethodView.java 100% <ø> (ø) 12 <0> (?)
...transformer/py/PythonImportSectionTransformer.java 97.14% <100%> (ø) 60 <2> (?)
...pi/codegen/transformer/ApiCallableTransformer.java 96.61% <100%> (ø) 34 <0> (?)
...n/transformer/DynamicLangApiMethodTransformer.java 99.29% <100%> (ø) 27 <0> (?)
...en/transformer/StaticLangApiMethodTransformer.java 95.8% <100%> (ø) 59 <0> (?)
...gen/transformer/HeaderRequestParamTransformer.java 83.33% <83.33%> (ø) 7 <7> (?)

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 41ce919...95cb264. Read the comment docs.

@evaogbe
Copy link
Contributor Author

evaogbe commented Oct 30, 2017

@lukesneeringer @vam-google PTAL

import java.util.List;

public class HeaderRequestParamTransformer {
public List<HeaderRequestParamView> generateHeaderRequestParams(MethodContext context) {

This comment was marked as spam.

This comment was marked as spam.

Copy link
Contributor

@lukesneeringer lukesneeringer left a comment

Choose a reason for hiding this comment

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

One bug; consider it approved once that is fixed. :-)

@@ -329,3 +345,25 @@
{@param.keyName}={@param.name}
@end
@end

@private routingHeader(params)
b'{@routingHeaderFmtStr(params)}'.format({@routingHeaderFmtArgs(params)})

This comment was marked as spam.

return self._create_book(request, retry=retry, timeout=timeout)

metadata = [
(b'x-google-request-params',

This comment was marked as spam.

@evaogbe evaogbe merged commit 6d55c9c into googleapis:master Nov 11, 2017
@evaogbe evaogbe deleted the python-routing-header branch November 11, 2017 02:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants