-
-
Notifications
You must be signed in to change notification settings - Fork 916
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
Authorization update #3077
Authorization update #3077
Conversation
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.
Only cosmetic changes suggestions.
Co-authored-by: Ivan Maximov <sungam3r@yandex.ru>
Co-authored-by: Ivan Maximov <sungam3r@yandex.ru>
Co-authored-by: Ivan Maximov <sungam3r@yandex.ru>
5.1.1 or 5.2.0 ? |
5.1.1 since new APIs may be considered as a way to fix a bug |
I agree |
Codecov Report
@@ Coverage Diff @@
## master #3077 +/- ##
==========================================
+ Coverage 84.31% 84.38% +0.06%
==========================================
Files 360 360
Lines 15733 15830 +97
Branches 2556 2576 +20
==========================================
+ Hits 13266 13358 +92
- Misses 1857 1858 +1
- Partials 610 614 +4
Continue to review full report at Codecov.
|
Closes #3076
This also fixes the attribute so that it applies the metadata to query arguments or enum definitions also, if it is marked on those type of objects.