Fix bad openapi.yaml merge #6079
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #6079 +/- ##
=============================================
+ Coverage 73.56% 74.41% +0.85%
- Complexity 5434 5639 +205
=============================================
Files 381 386 +5
Lines 19805 20206 +401
Branches 2046 2088 +42
=============================================
+ Hits 14570 15037 +467
+ Misses 4212 4170 -42
+ Partials 1023 999 -24
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
There was a problem hiding this comment.
Thanks Kathy! Now, please permit me to prosthelytize regarding our big hotfix merges: IMHO, they are one of the most likely points in our process where defects might be introduced, and some may be hard to detect (unlike this one). That's one of the reasons why I lobby for keeping as much development as possible out of hotfixes and on develop.
Description
This PR fixes the build that was broken from merging the 1.16.6 hotfix into develop. The openapi.yaml had duplicate TOIL enums.
Review Instructions
develop builds should pass
Issue
n/a
Security and Privacy
If there are any concerns that require extra attention from the security team, highlight them here and check the box when complete.
e.g. Does this change...
Please make sure that you've checked the following before submitting your pull request. Thanks!
mvn clean install@RolesAllowedannotation