fix: add warnings for duplicated or conflicting type hints in bigfram…

[jialuoo]

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

• Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
• Ensure the tests and linter pass
• Code coverage does not decrease (if any source code was changed)
• Appropriate docs were updated (if necessary)

Fixes b/409565963.

[TrevorBergeron]

Maybe we should just warn if the decorator actually conflicts with the signature?

[jialuoo]

Maybe we should just warn if the decorator actually conflicts with the signature?

I considered that. While warning only on a signature conflict is an option, I'm concerned about the risk of users defining type hints in both the decorator and the function signature. Since this is just a warning and won't fail the execution, I lean toward warning users about any duplicated type hints too. What do you think?

[jialuoo]

Maybe we should just warn if the decorator actually conflicts with the signature?

I considered that. While warning only on a signature conflict is an option, I'm concerned about the risk of users defining type hints in both the decorator and the function signature. Since this is just a warning and won't fail the execution, I lean toward warning users about any duplicated type hints too. What do you think?

Per off-line discussion, we'll warn only the conflict.