Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(oas): handling header parameters options #136

Merged
merged 6 commits into from
Feb 21, 2022

Conversation

pmstss
Copy link
Contributor

@pmstss pmstss commented Feb 17, 2022

fixes #120

@pmstss pmstss added the Status: wip The PR is a WIP. label Feb 17, 2022
@pmstss pmstss self-assigned this Feb 17, 2022
@codeclimate
Copy link

codeclimate bot commented Feb 17, 2022

Code Climate has analyzed commit 5d7e3cf and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 89.2% (0.1% change).

View more on Code Climate.

@pmstss pmstss force-pushed the fix-#120/header_param_options branch 2 times, most recently from 55ea8f4 to 432f153 Compare February 18, 2022 08:45
@pmstss pmstss removed the Status: wip The PR is a WIP. label Feb 18, 2022
@pmstss pmstss marked this pull request as ready for review February 18, 2022 08:46
@pmstss pmstss force-pushed the fix-#120/header_param_options branch from 432f153 to 6cc2b8b Compare February 21, 2022 15:01
@pmstss pmstss requested a review from derevnjuk February 21, 2022 15:44
@pmstss pmstss force-pushed the fix-#120/header_param_options branch from 81e4d48 to 230cc28 Compare February 21, 2022 17:41
@pmstss
Copy link
Contributor Author

pmstss commented Feb 21, 2022

@derevnjuk now rebased)

@pmstss pmstss requested a review from derevnjuk February 21, 2022 18:09
@derevnjuk derevnjuk merged commit 59761a9 into master Feb 21, 2022
@derevnjuk derevnjuk deleted the fix-#120/header_param_options branch February 21, 2022 18:19
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.

Rely on style and explode options while serializing headers
2 participants