support NewType

[pydantic]

support NewType

[samuelcolvin]

Surely simpler to construct the class properly:

class BossId(str):
    pass

That way you can easily add custom validation for BossIds etc.

If you really don't want to do that, why not use:

BossId = type('BossId', str, {})

?

If for some reason this really doesn't work, I'd be happy to accept a pull request for your fix above. But we might also need extra logic so that DirectReport.boss is an instance of BossId not just a str. How would this work?

[TimSimpsonR]

Hmm, good point that class Type(str) could be a valid work around. I might consider that for string types that require validation or if I need the type info to be attached to the value at runtime.

NewType is designed to be cheap; while MyPy validates it, at runtime it's only a function call, and the instances it creates are the underlying type. So I think it would be ok if Pydantic treated BossId as just a str too.

[samuelcolvin]

humm, ok but I bet you someone in future will come back and whinge when they get confused about the type of DirectReport.boss.

Since your change above should have no effect on performance for other usage I'd be happy to accept a pull request for this change provided it has full coverage and documentation.

[samuelcolvin]

Happy to accept a PR which supports this provided it's not to complicated.

CustomerId should be validated with the standard str validation logic.