-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
v0.9.0 #128
Merged
Merged
v0.9.0 #128
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #128 +/- ##
==========================================
+ Coverage 99.76% 99.77% +0.01%
==========================================
Files 146 146
Lines 10094 10174 +80
==========================================
+ Hits 10070 10151 +81
+ Misses 24 23 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Brackets in names are now supported. For example `’#/components/schemas/CoffeeCup[Black]’ Signed-off-by: Dave Shanley <dave@quobix.com>
Also ran `gofmt` across the entire project. Things need cleaning up. Signed-off-by: Dave Shanley <dave@quobix.com>
An error is thrown if both URL and Identifier are used together as they are mutually exclusive. Signed-off-by: Dave Shanley <dave@quobix.com>
error handling is still in the low level model. Signed-off-by: Dave Shanley <dave@quobix.com>
Signed-off-by: Dave Shanley <dave@quobix.com>
There is now a distinction between no security being defined by operations, and empty security requirements. If the `Security` value is `nil`, then it was left out. If the slice is empty, it was defined and left empty by design. As requested by #111 Signed-off-by: Dave Shanley <dave@quobix.com>
Original input document types are now rendered automatically as JSON or YAML (vs always being YAML). This only operates at the `Document` level for rendering out entire documents. Requested in #105 Signed-off-by: Dave Shanley <dave@quobix.com>
Fixing some outdated code as well. Signed-off-by: Dave Shanley <dave@quobix.com> Cleaning up more coverage issues Signed-off-by: Dave Shanley <dave@quobix.com> More test coverage updates. Signed-off-by: Dave Shanley <dave@quobix.com>
sometimes it does not make any sence, as the patch coverage is down, but overall coverage is up. With such small fractions, it’s causing more issues than anything else. Dropping down to manual coverage inspection to keep pipelines green with good builds. Signed-off-by: Dave Shanley <dave@quobix.com>
This is to allow the correct operation of local/remote lookups to be correctly indexed (after the resolver has run). Addresses an issue in vacuum daveshanley/vacuum#294 Signed-off-by: Dave Shanley <dave@quobix.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixing bugs and some breaking changes to the schema.
Provides fixes for:
identifier
field in OpenAPI 3.1 documents #124Will also include: