fix case of checking truthiness of NotImplemented

[davidhewitt]

Change Summary

At the moment when checking for non-negative integers, we can get false-postives from types that aren't integer.

e.g. the original logic was equivalent to

>>> bool("__all__".__lt__(0)) <stdin>:1: DeprecationWarning: NotImplemented should not be used in a boolean context True

... because comparisons between strings and integers are not implemented, and the dunder method does not raise type errors on NotImplemented.

Related issue number

Fixes pydantic/pydantic#9328

Checklist

• Unit tests for the changes exist
• Documentation reflects the changes where applicable
• Pydantic tests pass with this pydantic-core (except for expected changes)
• My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

[sydney-runkle]

Thanks!

[codspeed-hq]

CodSpeed Performance Report

Merging #1298 will not alter performance

_{Comparing dh/negative-comparison (293f4c3) with main (2415f42)}

Summary

✅ 155 untouched benchmarks

[MatthieuSarter]

Thanks for the fix 👍