Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Separate warnings and errors in Open API YAML reader ReadAsync method#1006

Merged
millicentachieng merged 3 commits intovnextmicrosoft/OpenAPI.NET:vnextfrom
ma/readasync-filter-errorsmicrosoft/OpenAPI.NET:ma/readasync-filter-errorsCopy head branch name to clipboard
Sep 9, 2022
Merged

Separate warnings and errors in Open API YAML reader ReadAsync method#1006
millicentachieng merged 3 commits intovnextmicrosoft/OpenAPI.NET:vnextfrom
ma/readasync-filter-errorsmicrosoft/OpenAPI.NET:ma/readasync-filter-errorsCopy head branch name to clipboard

Conversation

@millicentachieng
Copy link
Contributor

Fixes #1003

src/Microsoft.OpenApi.Readers/OpenApiYamlDocumentReader.cs Outdated Show resolved Hide resolved
src/Microsoft.OpenApi.Readers/OpenApiYamlDocumentReader.cs Outdated Show resolved Hide resolved
irvinesunday
irvinesunday previously approved these changes Sep 9, 2022
@irvinesunday
Copy link
Contributor

Bump the version of the Microsoft.OpenApi.Readers

baywet
baywet previously approved these changes Sep 9, 2022
@millicentachieng millicentachieng dismissed stale reviews from baywet and irvinesunday via 6cdceb0 September 9, 2022 13:39
@millicentachieng millicentachieng merged commit 0b710d5 into vnext Sep 9, 2022
@millicentachieng millicentachieng deleted the ma/readasync-filter-errors branch September 9, 2022 13:59
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.

ReadAsync method does not filter errors into errors and warnings as the Read method does

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.