8000 update docs for bool to match code by trim21 · Pull Request #911 · pydantic/pydantic · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@trim21
Copy link
Contributor
@trim21 trim21 commented Oct 17, 2019

Change Summary

Update docs for usage/types/#booleans

a str which when converted to lower case is one of 'off', 'f', 'false', 'n', 'no', '1', 'on', 't', 'true', 'y', 'yes'

Missing value '0' in code

https://github.com/samuelcolvin/pydantic/blob/ed7b216e5fe436756ff61e3a2dfed63aa8836464/pydantic/validators.py#L77

Related issue number

No

Checklist

  • [ ] Unit tests for the changes exist not changing any code.
  • [ ] Tests pass on CI and coverage remains at 100% not changing any code.
  • Documentation reflects the changes where applicable
  • changes/<pull request or issue id>-<github username>.md file added describing change

@codecov
Copy link
codecov bot commented Oct 17, 2019

Codecov Report

Merging #911 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #911   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          16     16           
  Lines        2751   2751           
  Branches      523    523           
=====================================
  Hits         2751   2751

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed7b216...de2c2bf. Read the comment docs.

@samuelcolvin samuelcolvin merged commit afba60f into pydantic:master Oct 17, 2019
@samuelcolvin
Copy link
Member

great, thanks.

andreshndz pushed a commit to cuenca-mx/pydantic that referenced this pull request Jan 17, 2020
* update docs for bool to match code

* add changelog
alexdrydew pushed a commit to alexdrydew/pydantic that referenced this pull request Dec 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0