Skip to content

Conversation

@devalog
Copy link
Collaborator

@devalog devalog commented Jul 3, 2025

Fill out configuration.mdx file for the Python SDK based on this README (just the SDK Generator Configuration section) and the Python config class

I didn't add client_class_name and client_filename to this file because they're marked as deprecated. I also think use_typeddict_requests_for_file_upload should be removed based on its description but would like confirmation!

One additional note –skip_formatting defaults to false per the config class, but readme says default is true. I put false in the file.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

- name: fernapi/fern-python
version: 0.6.6
config:
include_validators: true
Copy link
Member

Choose a reason for hiding this comment

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

can you show can example of using the client.name property since that is the most used config!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes! would that look like this?

config:
  client:
    filename: "my_custom_client.py"
    class_name: "MyClient"
    exported_filename: "my_client.py"
    exported_class_name: "APIClient"

and actually, the subfields for the client option aren't documented on this page. something to add?

@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

filename: my_custom_client.py
class_name: MyClient
exported_filename: my_client.py
exported_class_name: APIClient
Copy link
Member

Choose a reason for hiding this comment

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

@devalog can you try

client:
  class_name: `YourClient` 

@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

@devalog devalog merged commit f728e21 into main Jul 3, 2025
1 check 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.

3 participants