Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

feat: support non-slash resource in path template#833

Merged
alexander-fenster merged 3 commits intomasterfrom
support-non-slash
May 21, 2020
Merged

feat: support non-slash resource in path template#833
alexander-fenster merged 3 commits intomasterfrom
support-non-slash

Conversation

@xiaozhenliu-gg5
Copy link
Copy Markdown
Contributor

we allow non-slash resource now, for example:

user/{user_id}/blurbs/legacy/{blurb_id}.{legacy_user}'

@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2020

Codecov Report

Merging #833 into master will increase coverage by 0.10%.
The diff coverage is 81.08%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #833      +/-   ##
==========================================
+ Coverage   89.57%   89.68%   +0.10%     
==========================================
  Files          46       46              
  Lines        7408     7426      +18     
  Branches      548      512      -36     
==========================================
+ Hits         6636     6660      +24     
+ Misses        769      763       -6     
  Partials        3        3              
Impacted Files Coverage Δ
src/pathTemplate.ts 89.41% <81.08%> (+1.64%) ⬆️
.mocharc.js 78.57% <0.00%> (-14.29%) ⬇️
src/normalCalls/retries.ts 98.71% <0.00%> (+5.12%) ⬆️

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 392a392...4cda67c. Read the comment docs.

@alexander-fenster alexander-fenster merged commit 76696fc into master May 21, 2020
@alexander-fenster alexander-fenster deleted the support-non-slash branch May 21, 2020 17:56
gcf-merge-on-green Bot pushed a commit that referenced this pull request May 21, 2020
🤖 I have created a release \*beep\* \*boop\* 
---
## [2.4.0](https://www.github.com/googleapis/gax-nodejs/compare/v2.3.1...v2.4.0) (2020-05-21)


### Features

* parse path template using regexes ([#823](https://www.github.com/googleapis/gax-nodejs/issues/823)) ([392a392](https://www.github.com/googleapis/gax-nodejs/commit/392a3920df6d78981ac43741f15048c84102b046))
* support non-slash resource in path template ([#833](https://www.github.com/googleapis/gax-nodejs/issues/833)) ([76696fc](https://www.github.com/googleapis/gax-nodejs/commit/76696fc48c8a5e21c3c1cde56822b7a37585e41c))


### Bug Fixes

* new typescript, strict types ([#824](https://www.github.com/googleapis/gax-nodejs/issues/824)) ([90034ce](https://www.github.com/googleapis/gax-nodejs/commit/90034ce6a8c9b635942fedb23345105264979416))
* typescript 3.9.3 compilation ([#831](https://www.github.com/googleapis/gax-nodejs/issues/831)) ([d53e169](https://www.github.com/googleapis/gax-nodejs/commit/d53e16988aa0fa260c91f84e6fc0ceae2fdecc26))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
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