Skip to content

Conversation

samuelcolvin
Copy link
Member

@samuelcolvin samuelcolvin commented Dec 27, 2018

Change Summary

Rename get_validators to __get_validators__ on custom classes to be more like dataclasses (ref #334), also because it's a more sensible name.

Raise a depreciation warning on use of the old name.

We'll remove get_validators usage and warning in future.

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes
  • No performance deterioration (if applicable)
  • HISTORY.rst has been updated
    • if this is the first change since a release, please add a new section
    • include the issue number or this pull request number #<number>
    • if you're not a regular contributer please include your github username @whatever

@codecov
Copy link

codecov bot commented Dec 27, 2018

Codecov Report

Merging #338 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #338   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          13     13           
  Lines        1801   1828   +27     
  Branches      350    369   +19     
=====================================
+ Hits         1801   1828   +27

@samuelcolvin samuelcolvin merged commit 9ad1a0a into master Dec 27, 2018
@samuelcolvin samuelcolvin deleted the rename-get_validators branch December 27, 2018 20:30
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.

1 participant