Justin Chu

Strange: https://github.com/microsoft/onnxruntime/actions/runs/3099926945/jobs/5019640574#step:8:39 clang-tidy failed with return code 134 and error: LLVM ERROR: Cannot chdir into "/home/runner/work/onnxruntime/onnxruntime/build/Debug"!

@edgchen1 @cloudhan should modernize-use-nodiscard be disabled? Otherwise this is ready to merge. 

@edgchen1 PTAL

@cloudhan PTAL

Love this and thank you for your response! In the function you shared, I noticed that a warning is emitted (great!) but the function was not run. We were looking...

I can try to prototype this in a PR. My current plan is to see if I can condition on where the roars happen and warn + return instead. Do...

Wonderful! That's probably good enough for our use now. To make sure it works in all cases, does beartype check for return types as well? If it does, I wonder...

This is fantastic, thank you so much for this! 😄 I played with the snippet and settled on what's below, where I decided the return type is not so important...

Looks like adding hint_return back causes k to become 3 in ``` k = 0 @bearcubtype def spell(x: str) -> int: global k k += 1 return str(x) print(spell("x")) print(spell(42))...

It does show 2 now. My bad!