-
Notifications
You must be signed in to change notification settings - Fork 3
Fix unstable API documentation issues #291
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
Conversation
59b9cd3 to
042a9a8
Compare
Map snake_case schema names to PascalCase for CreateDataAttributeRequest and UpdateDataAttributeRequest to resolve type definition errors in fern check. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
EDIT: I see you have linked the previous PR |
| @@ -1,4 +1,18 @@ | |||
| paths: | |||
| '/data_attributes': | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we sure we want to make these changes here 🤔
I have low context on fern, but this looks like it was pushed by someone from fern
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these are just config files used to generate SDKs 🤔 I needed to edit them as the build of the pr was failing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right, I asked cursor about that file:
When this configuration is processed by Fern:
- Reads the OpenAPI specifications from the descriptions/ directory
- Applies any overrides specified in the override files
- Generates complete SDK code for each language
- Creates pull requests in the respective GitHub repositories
- Publishes packages to their respective registries (NPM, Maven, PyPI)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also got it confirmed by @Eclairemoy on Slack 🙏🏻
VarshSuresh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
![]()
Summary
Test plan
🤖 Generated with Claude Code