Skip to content

Method name is added to exception message. - #787

Merged
Shawn Callegari (shawncal) merged 2 commits into
microsoft:mainfrom
SergeyMenshykh:improve-exception-message
May 6, 2023
Merged

Method name is added to exception message.#787
Shawn Callegari (shawncal) merged 2 commits into
microsoft:mainfrom
SergeyMenshykh:improve-exception-message

Conversation

@SergeyMenshykh

Copy link
Copy Markdown
Contributor

Motivation, Context and Description

This PR adds function/method name into exception message that is thrown if a native function, marked with SKFunctionInputAttribute, doesn't have an input parameter of string type. The method name can be useful for debugging and troubleshooting.

Contribution Checklist

@SergeyMenshykhSergeyMenshykh added the PR: ready for review All feedback addressed, ready for reviews label May 3, 2023
@github-actionsgithub-actionsBot added .NET Issue or Pull requests regarding .NET code kernel.core labels May 3, 2023
Comment threaddotnet/src/SemanticKernel/SkillDefinition/SKFunction.cs
@shawncal
Shawn Callegari (shawncal) enabled auto-merge (squash) May 6, 2023 05:35
@shawncal
Shawn Callegari (shawncal) merged commit 22d12cb into microsoft:mainMay 6, 2023
Stuart Cam (codebrain) pushed a commit to searchpioneer/semantic-kernel that referenced this pull request May 16, 2023
### Motivation, Context and Description
This PR adds function/method name into exception message that is thrown
if a native function, marked with SKFunctionInputAttribute, doesn't have
an input parameter of string type. The method name can be useful for
debugging and troubleshooting.
Desmond Howard (dehoward) pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
### Motivation, Context and Description
This PR adds function/method name into exception message that is thrown
if a native function, marked with SKFunctionInputAttribute, doesn't have
an input parameter of string type. The method name can be useful for
debugging and troubleshooting.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NETIssue or Pull requests regarding .NET codePR: ready for reviewAll feedback addressed, ready for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@SergeyMenshykh@dluc@shawncal@adrianwyatt