By default, be picker about callable

[liamhuber]

typeguard.check_type is quite relaxed when seeing if something is callable -- so much so that everything will pass through, even check_type(5, callable), or (as in the test suite) a check against an instance of a class that has not defined __call__. This switches the default behaviour of type_hinting.valid_value to convert raw callable hints into collections.abc.Callable, which check_type treats more rigorously. I believe this is closer to the intuitively expected behaviour of valid_value, but the original behaviour can still be recovered via a flag argument.

@XzzX, I only noticed this because of #545, so thanks for pushing that!

[github-actions]

👈 Launch a binder notebook on branch pyiron/pyiron_workflow/stricter_callable

[coveralls]

Pull Request Test Coverage Report for Build 12795803361

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+0.002%) to 91.48%

---

Files with Coverage Reduction	New Missed Lines	%
type_hinting.py	1	97.78%

Totals
Change from base Build 12795134771:	0.002%
Covered Lines:	3071
Relevant Lines:	3357

---

💛 - Coveralls