wrap_json and wrap_constant type fixes

[henadzit]

• Removed unused ignores for wrap_constant
• Simplify typing of wrap_json
• Make Interval a subclass of Term. It doesn't seem like there is reason it should be a subclass of Node. This helps with simplifying typing.
• Set warn_unused_ignores = true for mypy and remove unused ignores

[waketzheng]

Great! Looks nice, the only thing is that Interval is not a subclass of Term, please check it.

[henadzit]

Great! Looks nice, the only thing is that Interval is not a subclass of Term, please check it.

Could you please elaborate on what you mean? I couldn't figure out why the Interval should be a subclass of Node in particular. It seems to work just fine with it being a subclass of Term.

[waketzheng]

https://github.com/tortoise/pypika-tortoise/pull/21/files#diff-54340751c88780ef96f3ba11bb105a16d92a43a583141992cc7e4162507ac013L107

You change if isinstance(val, (Term, QueryBuilder, Interval)): to if isinstance(val, Term):
QueryBuilder is subclass of Term, while Interval is not. So these two line have different logic.

[waketzheng]

Please fix conflicts