Skip to content

Conversation

FacerAin
Copy link
Contributor

@FacerAin FacerAin commented Aug 30, 2023

Change Summary

In this migration docs, it seems desirable to change "pydantic-settings" to "pydantic_settings" with underscores.
Even though the package repo name is "pydantic-settings", In Python, we still need to use underscores as shown below.

from pydantic_settings import BaseSettings

I believe this change will be helpful for users.

Thanks! ☺️

Related issue number

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @adriangb

@FacerAin
Copy link
Contributor Author

please review

@hramezani
Copy link
Member

Thanks @FacerAin

@hramezani hramezani merged commit b9d2a98 into pydantic:main Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants