Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Document alias_priority #6520

Merged
merged 1 commit into from Jul 8, 2023
Merged

Document alias_priority #6520

merged 1 commit into from Jul 8, 2023

Conversation

tpdorsey
Copy link
Contributor

@tpdorsey tpdorsey commented Jul 7, 2023 •
edited by pydantic-hooky bot

Documentation added for alias_priority in Fields and Model Config.

skip change file check

Change Summary

Related issue number

Removes a TODO in the docs.

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes where applicable
  • changes/<pull request or issue id>-<github username>.md file added describing change
    (see changes/README.md for details)
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @lig

@tpdorsey
Copy link
Contributor Author

tpdorsey commented Jul 7, 2023

please review

@Kludex Kludex merged commit c5b28b8 into main Jul 8, 2023
50 checks passed
@Kludex Kludex deleted the alias-priority branch July 8, 2023 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kludex Kludex Kludex approved these changes

Assignees

@lig lig

Labels
documentation ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@tpdorsey @Kludex @lig