Skip to content

Conversation

rnazali
Copy link
Contributor

@rnazali rnazali 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.

@rnazali
Copy link
Contributor Author

rnazali commented Oct 19, 2023

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
@rnazali rnazali 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.

3 participants