feat: Add more details to MissingTargetException error#158
Conversation
KaylaNguyen
left a comment
KaylaNguyen
left a comment
There was a problem hiding this comment.
Thank you for working on this! I'll approve it once it's up-to-date.
| @@ -256,9 +256,15 @@ def handle_none(rv): | |||
|
|
|||
| # Extract the target function from the source file | |||
| if not hasattr(source_module, target): | |||
There was a problem hiding this comment.
Would you mind syncing this PR to main branch? This line should now be in _function_registry.py.
There was a problem hiding this comment.
@KaylaNguyen I'm not a maintainer of this repo anymore, and thus don't have write access to this branch.
There was a problem hiding this comment.
Hi Dustin, would you mind forking this repo and copy this branch to a new one on di/functions-framework-python and create a PR from that instead? Then you won't need to be a maintainer of the repo to make changes to the branch. We can add you as maintainer of this repo to proceed with these 2 commits but all admins are OOO atm.
3 participants
Include names of other functions found in the file. This will help debug issues like #157.