Skip to content

Fix error in openapi conversion #101

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

Merged
merged 1 commit into from
Apr 23, 2025

Conversation

danlapid
Copy link
Contributor

@danlapid danlapid commented Apr 22, 2025

Fixes: UnboundLocalError: cannot access local variable 'param_desc' where it is not associated with a value.

This is happening after 4ad4eef because param_desc is read without being defined.

Usage at https://github.com/tadata-org/fastapi_mcp/pull/101/files#diff-95219e0f955edb24aeee85fc762c170b6eca135e23b00ecc167c8744620e1eddR244

Describe your changes

Issue ticket number and link (if applicable)

Screenshots of the feature / bugfix

Checklist before requesting a review

  • Added relevant tests
  • Run ruff & mypy
  • All tests pass

Fixes: UnboundLocalError: cannot access local variable 'param_desc' where it is not associated with a value.

This is happening after tadata-org@4ad4eef because param_desc is read without being defined.
Copy link

codecov bot commented Apr 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@shahar4499 shahar4499 merged commit dc4a9d4 into tadata-org:main Apr 23, 2025
6 checks passed
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