-
Notifications
You must be signed in to change notification settings - Fork 66
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
refactor: use swagger for table model in search #110
Conversation
8a9dd7d
to
ffce8be
Compare
Codecov Report
@@ Coverage Diff @@
## master #110 +/- ##
==========================================
+ Coverage 75.20% 75.28% +0.08%
==========================================
Files 17 17
Lines 613 615 +2
Branches 76 77 +1
==========================================
+ Hits 461 463 +2
Misses 127 127
Partials 25 25
Continue to review full report at Codecov.
|
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.
Looks good.
setup.py
Outdated
@@ -2,7 +2,7 @@ | |||
|
|||
from setuptools import setup, find_packages | |||
|
|||
__version__ = '2.3.3' | |||
__version__ = '2.3.4rc0' |
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.
Don't forget to remove rc0
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.
will test on staging first :)
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.
works
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.
LGTM
Summary of Changes
Use swagger for table model in search to make it consistent.
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.
make test