-
Notifications
You must be signed in to change notification settings - Fork 84
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
Added enum custom field names option #1015
Conversation
Precisely what I was looking for, thank you! Could be interesting to also try to support the OAS 3.1 recommended syntax MyEnum
type: integer
oneOf:
- title: First
const: 1
description: "First value of my enum"
- title: Second
const: 2
description: "Second value of my enum" |
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.
Could you reword commits to follow Conventional Commits, please?
Also, it seems some CI tests failed.
a5c964a
to
7a89350
Compare
6f1a75a
to
12fc8e3
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1015 +/- ##
==========================================
+ Coverage 72.20% 72.22% +0.02%
==========================================
Files 192 192
Lines 15027 15040 +13
==========================================
+ Hits 10850 10863 +13
- Misses 3641 3642 +1
+ Partials 536 535 -1
☔ View full report in Codecov by Sentry. |
Unfortunately I don't have the time to invest in getting this PR "to code", if anyone wants to pick it up, please do. |
Useful for non-string enums.
Also added
String()
impl for such enums.