Skip to content

Conversation

gr2m
Copy link
Contributor

@gr2m gr2m commented Oct 19, 2020

fixes #329

@codecov
Copy link

codecov bot commented Oct 20, 2020

Codecov Report

Merging #337 into master will increase coverage by 4.31%.
The diff coverage is 90.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #337      +/-   ##
==========================================
+ Coverage   86.40%   90.71%   +4.31%     
==========================================
  Files           5        5              
  Lines         250      237      -13     
  Branches       88       81       -7     
==========================================
- Hits          216      215       -1     
+ Misses         22       16       -6     
+ Partials       12        6       -6     
Impacted Files Coverage Δ
src/index.ts 81.81% <60.00%> (-4.55%) ⬇️
src/v2.ts 83.72% <81.57%> (-2.95%) ⬇️
src/v3.ts 92.59% <92.30%> (+24.29%) ⬆️
src/property-mapper.ts 84.21% <100.00%> (+5.26%) ⬆️
src/utils.ts 100.00% <100.00%> (+2.56%) ⬆️

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 624b66c...8fd72df. Read the comment docs.

@gr2m gr2m marked this pull request as ready for review October 20, 2020 03:59
if (p.name) {
allParameters[p.in][p.name] = p;
if ("$ref" in p) {
// return;
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done in #340

query: {
per_page: components["parameters"]["per_page"];
page: components["parameters"]["page"];
since: components["parameters"]["since"];
Copy link
Contributor

Choose a reason for hiding this comment

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

💯

);
});

it("$ref-type parameters (#329)", () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

💯 Great tests!

Copy link
Contributor

@drwpow drwpow left a comment

Choose a reason for hiding this comment

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

This looks great! I think there’s just a conflict from one of your other PRs.

Thanks for the great addition 🙇🏻

@drwpow drwpow closed this in #340 Oct 23, 2020
drwpow referenced this pull request Oct 23, 2020
* test: `$ref`-type parameters

* feat: `$ref`-type parameters

* test: `requestBody` types

* feat: `requestBody` types

* refactor: remove obsolete line of code

addresses https://github.com/drwpow/openapi-typescript/pull/337\#discussion_r511181303
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support $ref-type parameters, as well as components.parameters
2 participants