Skip to content

fix: OpenAPI generated samples are displayed out of order#328

Merged
b-long merged 1 commit into
mainfrom
fix/openapi-ordering
May 20, 2026
Merged

fix: OpenAPI generated samples are displayed out of order#328
b-long merged 1 commit into
mainfrom
fix/openapi-ordering

Conversation

@b-long
Copy link
Copy Markdown
Contributor

@b-long b-long commented May 20, 2026

Fixes #327

Summary by CodeRabbit

  • Chores
    • Reorganized the order of programming language options in API documentation tabs for improved usability.

Review Change Stack

@b-long b-long requested review from a team as code owners May 20, 2026 18:27
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 82c536d5-5009-41bd-af25-ba92869d0d3c

📥 Commits

Reviewing files that changed from the base of the PR and between dfc9028 and 65d14f9.

📒 Files selected for processing (1)
  • openapi-generated-clients.ts

📝 Walkthrough

Walkthrough

The languageTabs array in openapi-generated-clients.ts is reorganized across two segments, rearranging the order of language options (C, C#, Curl, Dart, Kotlin, PHP, PowerShell, Python, R, Ruby, Rust, Swift) for OpenAPI-generated code samples.

Changes

Language Tabs Reordering

Layer / File(s) Summary
Language tabs reordering
openapi-generated-clients.ts
The languageTabs configuration array is reorganized to display code sample language options in a revised sequence, updating two separate segments of the array with rearranged language tab objects.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Lines reordered neat and fine,
Language tabs in grand align!
From C to Swift, a cleaner way,
Code samples shine throughout the day! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: fixing the ordering of OpenAPI generated code samples.
Linked Issues check ✅ Passed The PR successfully addresses issue #327 by reordering the languageTabs array to display samples in alphabetical order instead of the previous inconsistent sequence.
Out of Scope Changes check ✅ Passed All changes are directly related to fixing the language tab ordering issue; no unrelated modifications were introduced.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/openapi-ordering

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@marythought marythought left a comment

Choose a reason for hiding this comment

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

🔡

@b-long b-long merged commit 5be45e2 into main May 20, 2026
6 checks passed
@b-long b-long deleted the fix/openapi-ordering branch May 20, 2026 18:31
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request reorders the languageTabs array in openapi-generated-clients.ts, updating the display sequence for various programming languages such as C, Bash/Curl, Kotlin, PHP, and Python. I have no feedback to provide.

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.

OpenAPI generated samples are displayed out of order

2 participants