Skip to content
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

Sort object props/params by default, hoist required ones #55

Merged
merged 2 commits into from
Jun 27, 2024
Merged

Conversation

brotheroftux
Copy link
Contributor

Main changes:

  • Lexicographic sorting for endpoint params/object schema props is now ON by default
  • Required params/object schema props are hoisted to the top by default

Misc:

  • Changed the test config a bit — I think it makes sense to treat __tests__ as a folder for all integration tests, and unit tests should be stored close to the code they are testing (see presentationUtils folder structure)

@brotheroftux brotheroftux requested review from 3y3 and v8tenko June 16, 2024 16:41
@brotheroftux brotheroftux force-pushed the DOCSTOOLS-2255 branch 2 times, most recently from 18aab67 to ec6b6ab Compare June 16, 2024 16:51
@brotheroftux brotheroftux force-pushed the DOCSTOOLS-2255 branch 2 times, most recently from 6b3f250 to 455e336 Compare June 21, 2024 20:51
@3y3 3y3 merged commit 271fdcc into master Jun 27, 2024
2 checks passed
@3y3 3y3 deleted the DOCSTOOLS-2255 branch June 27, 2024 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants