-
-
Notifications
You must be signed in to change notification settings - Fork 159
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
Small refactor #1116
Small refactor #1116
Conversation
…e the vocabulary is 'RelationshipsObject' with its members being a 'Relationship'.
6dfefc4
to
6160ad0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added some questions.
Note the cibuild fails one test, which also happens in the openapi branch. This surprises me because, on my laptop, all tests succeed. The failure is due to the location of the nullable schema in swagger.json. Download the full log from AppVeyor to see the diff. Perhaps a timing issue. Can you take a look?
a277ff6
to
6160ad0
Compare
Codecov Report
@@ Coverage Diff @@
## openapi #1116 +/- ##
===========================================
+ Coverage 88.98% 88.99% +0.01%
===========================================
Files 298 308 +10
Lines 8117 8180 +63
===========================================
+ Hits 7223 7280 +57
- Misses 894 900 +6
Continue to review full report at Codecov.
|
da639f7
to
b527567
Compare
b527567
to
796f104
Compare
72391df
to
d877f7d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
TODO:
openapi
.QUALITY CHECKLIST