Skip to content

📝 Fix doctrings for max_digits and decimal_places#14944

Open
YuriiMotov wants to merge 1 commit intomasterfrom
fix-max_digits-docstring
Open

📝 Fix doctrings for max_digits and decimal_places#14944
YuriiMotov wants to merge 1 commit intomasterfrom
fix-max_digits-docstring

Conversation

@YuriiMotov
Copy link
Member

@YuriiMotov YuriiMotov added the docs Documentation about how to use FastAPI label Feb 19, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 19, 2026

Merging this PR will not alter performance

✅ 20 untouched benchmarks


Comparing fix-max_digits-docstring (c5e31ae) with master (faee822)1

Open in CodSpeed

Footnotes

  1. No successful run was found on master (c441583) during the generation of this report, so faee822 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@Areeb455

This comment was marked as resolved.

@YuriiMotov

This comment was marked as resolved.

@Areeb455

This comment was marked as resolved.

@YuriiMotov
Copy link
Member Author

YuriiMotov commented Feb 22, 2026

@Areeb455, this PR hasn't been merged yet. So, to check it you need to checkout to the branch of this PR (fix-max_digits-docstring).
If you do it, you will see that the docstrings are:
Maximum number of digits allowed for decimal values. and Maximum number of decimal places allowed for decimal values.

@Areeb455
Copy link

Thank you @YuriiMotov! I've reviewed the latest changes in the branch. The docstrings are now technically accurate and the wording is much clearer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation about how to use FastAPI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants