`result_is_success` should be inlined and should not use a set expression

[smarie]

result_is_success is currently not inlined
and it uses return validation_result in {None, True} which may fail if validation_result is not hashable, and is definitely slower than (validation_result is True) or (validation_result is None)