docs(openapi): document bigint response header - #1663
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (27)
📒 Files selected for processing (1)
WalkthroughThe change adds v2 API tests for ChangesBigInt serialization test coverage
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. 🔧 golangci-lint (2.12.2)level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain main module or its selected dependencies" 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. Comment |
✅ Approve — automated reviewThe PR consistently adds a reusable bigint response header parameter across the OpenAPI documents and regenerated client code, aligning with the existing bigint-as-string render paths. No correctness issues were identified by any reviewer, and the prior automated review also found nothing to flag. The changes are clean and ready to merge. No findings. |
|
Thank you for your contribution @shivanshgupta365 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1663 +/- ##
==========================================
+ Coverage 81.25% 81.42% +0.16%
==========================================
Files 206 206
Lines 11302 11298 -4
==========================================
+ Hits 9184 9199 +15
+ Misses 1560 1555 -5
+ Partials 558 544 -14 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Summary
Validation
Closes #1320