Skip to content

Conversation

@marcmlc
Copy link
Contributor

@marcmlc marcmlc commented Aug 14, 2025

Summary

Test plan

  • Documentation changes only
  • No functional changes

🤖 Generated with Claude Code

@marcmlc marcmlc force-pushed the marcmeillac/unstable-doc-fix branch from 59b9cd3 to 042a9a8 Compare August 14, 2025 12:11
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>
@marcmlc marcmlc requested a review from a team August 14, 2025 12:39
@VarshSuresh
Copy link
Contributor

VarshSuresh commented Aug 14, 2025

Do you want to link this to some issue so it is not lost in the pile of PRs?

EDIT: I see you have linked the previous PR

@@ -1,4 +1,18 @@
paths:
'/data_attributes':
Copy link
Contributor

@VarshSuresh VarshSuresh Aug 14, 2025

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

Copy link
Contributor Author

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

Copy link
Contributor

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)

Copy link
Contributor Author

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 🙏🏻

Copy link
Contributor

@VarshSuresh VarshSuresh left a comment

Choose a reason for hiding this comment

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

:shipit:

@marcmlc marcmlc merged commit 02568c7 into main Aug 14, 2025
8 checks passed
@marcmlc marcmlc deleted the marcmeillac/unstable-doc-fix branch August 14, 2025 16:33
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