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
support keep spec properties order #1939
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1939 +/- ##
==========================================
- Coverage 80.91% 80.84% -0.08%
==========================================
Files 38 38
Lines 7589 7627 +38
==========================================
+ Hits 6141 6166 +25
- Misses 965 972 +7
- Partials 483 489 +6
Continue to review full report at Codecov.
|
would it be possible to add a test demonstrating this behavior? |
|
This is used in the generation process right? |
Signed-off-by: Efi-Kofler <efi.kofler@tectonic.network>
Am I missing tests for codecov checks ? |
Codecov is always unhappy, it's not a required check |
I am using mixin tool to create a single file from multiple swagger specs. In a primary/final spec, I want to maintain the properties order. This option seems to be available only for "generate server" command. Can we make this available for mixin command as well? |
Can this work for |
It doesn't appear to but would also like to see this happen. |
This PR solves issue #125 (Keep object's parameters order as specified in swagger spec definition).
Before loading the spec , if argument--keep-spec-order exists , it will inject x-order attribute to each property which is equal to its position in the spec file.
This change is