Patch function

[anderson-joyle]

This is work in progress but I want to get some feedback from reviewers.
Some of my questions/concerns/comments:

• Two Patch implementations for now:

Patch(record, record)
Not the focus although it is going to be implemented anytime soon.

Patch(dataSource, record, arg1, arg2, ...)
The priority one.

• In order to update a record, I'm simply removing the old record (ICollection.Remove(T)) and adding a new version of it. Is there a better/prettier way to do this?
• I'm still working on the CheckInvocation implementations.
• I'm still working on how to handle errors.
• A new handler has been added for test units. Is this the way or is there a better approach?
• Any other major concerns?

[anderson-joyle]

@MikeStall about this binary compatibility issue, this is happening because Microsoft.PowerFx.PowerFxConfigExtensions.EnableCollectFunction was previously merged into main and now it has been removed due to the new EnableMutationFunctions method. Should I bring back the EnableCollectFunction method just for compatibility purposes?

---

In reply to: 1228934844

---

In reply to: 1228934844

---

In reply to: 1228934844

[MikeStall]

This breaking change is ok. We just introduced the function, PowerApps doesn't depend on it yet.

---

In reply to: 1228934844

[BruceHaley]

✔️ No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
✔️ No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
❌ 1 Binary Compatibility issues for Microsoft.PowerFx.Interpreter compared against version 0.2.2-preview.20220830-319453.

Details

MembersMustExist : Member 'Microsoft.PowerFx.PowerFxConfigExtensions.EnableCollectFunction(Microsoft.PowerFx.SymbolTable)' does not exist in the implementation but it does exist in the contract.

✔️ No Binary Compatibility issues for **Microsoft.PowerFx.LanguageServerProtocol.dll** ✔️ No Binary Compatibility issues for **Microsoft.PowerFx.Transport.Attributes.dll**

[MikeStall]

[LucGenetier]