Conversation
Issue prkumar#281 TODO tests
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #282 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 44 42 -2
Lines 2471 2448 -23
Branches 358 355 -3
=========================================
- Hits 2471 2448 -23 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
prkumar
left a comment
There was a problem hiding this comment.
@ehooo - Thanks for taking the time to make this change!
Adding this to Sequence will add support to only a few annotations, like @Query and @QueryMap. Sequence is responsible for applying a converter to each item in a list of values.
To add broad support for enums, we should add this to either the TypingConverter or the StandardConverter.
|
Hello @prkumar Thanks for your response, i just check it and i don't know where do you want to add this conversion. I see that the property Please let me know where do you think i could move it. |
Fixes #281 .
Changes proposed in this pull request:
Attention: @prkumar