Power-Fx icon indicating copy to clipboard operation
Power-Fx copied to clipboard

Unifying Remove function with PA

Open anderson-joyle opened this issue 1 year ago • 18 comments

Issue https://github.com/microsoft/Power-Fx/issues/2808.

anderson-joyle avatar Dec 13 '24 19:12 anderson-joyle

✅ No public API change. #Resolved

LucGenetier avatar Dec 13 '24 20:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 16 '24 17:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 17 '24 20:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 19 '24 12:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 19 '24 16:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 19 '24 17:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 26 '24 21:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 28 '24 01:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 30 '24 15:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 30 '24 20:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Dec 31 '24 15:12 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Jan 02 '25 23:01 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Jan 09 '25 15:01 LucGenetier

✅ No public API change. #Resolved

LucGenetier avatar Jan 19 '25 22:01 LucGenetier

#SETUP: PowerFxV1CompatibilityRules,EnableExpressionChaining,MutationFunctionsTestSetup,StronglyTypedBuiltinEnums

The "base" test Remove.txt already defines PowerFxCompatibilityRules, the cases in this file can be merged into the base one.


Refers to: src/tests/Microsoft.PowerFx.Core.Tests.Shared/ExpressionTestCases/Remove_V1Compact.txt:1 in 1bf13e3. [](commit_id = 1bf13e3cec8d5275ebcb0db9bc60dee3c18ab6e5, deletion_comment = False)

CarlosFigueiraMSFT avatar Jan 20 '25 19:01 CarlosFigueiraMSFT

✅ No public API change.

LucGenetier avatar Jan 23 '25 05:01 LucGenetier

@copilot move this to a new PR based on latest from main branch, merging the changes to the latest code.

anderson-joyle avatar Nov 05 '25 15:11 anderson-joyle

@anderson-joyle I've opened a new pull request, #2996, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot avatar Nov 05 '25 15:11 Copilot