Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Oct 16, 2025

Fix Python SDK configuration key from package-name to package_name

Summary

Updated Python SDK documentation to use the correct configuration key package_name (with underscore) in the config section instead of the incorrect package-name (with hyphen). This fix addresses a documentation bug reported in #sdk-dev.

Key distinction preserved:

  • output.package-name (hyphen) - remains unchanged, used for PyPI publishing location
  • config.package_name (underscore) - fixed in this PR, used for the actual Python package configuration

Files changed:

  • fern/products/sdks/overview/python/configuration.mdx - Updated 4 instances in the config section YAML example and ParamField definitions
  • fern/products/sdks/snippets/readme-options.mdx - Fixed anchor link to reference the corrected package_name field

Review & Testing Checklist for Human

  • Verify generator behavior: Confirm that the Python SDK generator actually accepts config.package_name (underscore) and not config.package-name (hyphen) by testing with a real configuration or checking the generator source code
  • Verify output section is correct: Double-check that output.package-name (hyphen) is still the correct format for PyPI publishing - I kept it as hyphen based on generators-yml-reference.mdx, but this should be verified
  • Test documentation links: Navigate to the Python configuration page and verify that the #package_name anchor link works correctly from readme-options.mdx
  • Check for missed instances: Scan for any other Python-specific documentation that might reference package-name in the context of the config section that I may have missed

Notes

The Python SDK configuration uses underscores, not hyphens. Updated:
- configuration.mdx: Fixed package-name to package_name in example and ParamField paths
- publishing-to-pypi.mdx: Fixed package-name to package_name in all YAML examples

Co-Authored-By: [email protected] <[email protected]>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Contributor

- Updated readme-options.mdx to reference package_name (underscore) in config section
- Reverted publishing-to-pypi.mdx output sections back to package-name (hyphen)
- The output section uses package-name (hyphen) while config section uses package_name (underscore)

Co-Authored-By: [email protected] <[email protected]>
@github-actions
Copy link
Contributor

@tjb9dc tjb9dc merged commit 6195346 into main Oct 16, 2025
1 check passed
@tjb9dc tjb9dc deleted the devin/1760642238-fix-python-package-name-config branch October 16, 2025 19:49
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