Skip to content

feat: improve design doc samples [INS-5526] - #8719

Merged
ryan-willis merged 2 commits into
developfrom
feat/design-doc-spec-example-improvements-INS-5526
May 19, 2025
Merged

feat: improve design doc samples [INS-5526]#8719
ryan-willis merged 2 commits into
developfrom
feat/design-doc-spec-example-improvements-INS-5526

Conversation

@ryan-willis

@ryan-willis ryan-willis commented May 14, 2025

Copy link
Copy Markdown
Contributor
improve-design-doc-samples.mp4
  • disables linting/formating checks if spec contents are empty
  • adds two new samples
    • Todo List API spec
    • Blank API spec

@ryan-willis
ryan-willis force-pushed the feat/design-doc-spec-example-improvements-INS-5526 branch from dea3203 to 58dee0b Compare May 14, 2025 15:37
@chayglass84

Copy link
Copy Markdown

Love it.

Can we fix the petstore spec not to be riddled with warnings while we're at it? Even just whatever ChatGPT does to it?

Other then that, LGTM

@ryan-willis
ryan-willis force-pushed the feat/design-doc-spec-example-improvements-INS-5526 branch 2 times, most recently from 8960bcd to 5d47beb Compare May 17, 2025 05:18
@ryan-willis
ryan-willis marked this pull request as ready for review May 17, 2025 05:19
@ryan-willis
ryan-willis force-pushed the feat/design-doc-spec-example-improvements-INS-5526 branch from 5d47beb to ace949e Compare May 18, 2025 23:06

@gatzjames gatzjames left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ryan-willis
ryan-willis merged commit 77b8ca4 into develop May 19, 2025
@ryan-willis
ryan-willis deleted the feat/design-doc-spec-example-improvements-INS-5526 branch May 19, 2025 18:45
cwangsmv pushed a commit that referenced this pull request Jun 10, 2025
* feat: improve design doc samples [INS-5526]

* update to latest petstore, fixing lint errors
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.

3 participants