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.

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

Fix invalid reference inside Fields/Immutability documentation #7870

Merged
merged 1 commit into from Oct 19, 2023

Conversation

rahmatnazali
Copy link
Contributor

@rahmatnazali rahmatnazali commented Oct 19, 2023

Change Summary

On Fields/Immutability Documentation, there is a broken markdown link that supposed to refer to the frozen dataclass documentation.

As the link is already well defined several lines below it, I propose to remove this back to plain text.

Related issue number

None so far.

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 (this PR only contains documentation-related change)
  • Tests pass on CI (awaiting label)
  • 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

…ain text.

As it is already being declared few lines below it.
@@ -651,7 +651,7 @@ See [Conversion Table](conversion_table.md) for more details on how Pydantic con

## Immutability

The parameter `frozen` is used to emulate the [frozen dataclass] behaviour. It is used to prevent the field from being
The parameter `frozen` is used to emulate the frozen dataclass behaviour. It is used to prevent the field from being
assigned a new value after the model is created (immutability).

See the [frozen dataclass documentation] for more details.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The correct reference is already mentioned down here.

@rahmatnazali
Copy link
Contributor Author

please review

@hramezani hramezani added the relnotes-ignore Omit this PR from the release notes. label Oct 19, 2023
@hramezani
Copy link
Member

Thanks @rahmatnazali

@hramezani hramezani merged commit 76861b0 into pydantic:main Oct 19, 2023
59 of 61 checks passed
@rahmatnazali rahmatnazali deleted the docs-immutability branch October 19, 2023 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review relnotes-ignore Omit this PR from the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants