-
Notifications
You must be signed in to change notification settings - Fork 308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expose void return type and add reserved UDF names #2412
Conversation
src/libraries/Microsoft.PowerFx.Core/Functions/UserDefinedFunction.cs
Outdated
Show resolved
Hide resolved
src/tests/Microsoft.PowerFx.Core.Tests/UserDefinedFunctionTests.cs
Outdated
Show resolved
Hide resolved
…uyen/udf-void-type-and-reserved-names
…thub.com/microsoft/Power-Fx into rnguyen/udf-void-type-and-reserved-names
✅ No public API change. |
src/libraries/Microsoft.PowerFx.Core/Functions/UserDefinedFunction.cs
Outdated
Show resolved
Hide resolved
src/libraries/Microsoft.PowerFx.Core/Functions/UserDefinedFunction.cs
Outdated
Show resolved
Hide resolved
I know this is not part of your changes, but does this comment still make sense? I couldn't find any definition for "PrimitiveTypeSymbolTable" Refers to: src/libraries/Microsoft.PowerFx.Core/Public/Types/FormulaType.cs:77 in ca62f6f. [](commit_id = ca62f6f, deletion_comment = False) |
✅ No public API change. |
src/libraries/Microsoft.PowerFx.Core/Functions/UserDefinedFunction.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ No public API change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With imperative functions, add support for a void UDF type. Also some functions should not be allowed for UDF names like Set, Collect, etc.