search

microsoft / Power-Fx

Power Fx low-code programming language
MIT License
3.2k stars 323 forks source link

Unify Collect /Patch texl functions with PA implementation #1550

Open MikeStall opened 1 year ago

MikeStall commented 1 year ago

The Mutation texlFunctions ( https://github.com/microsoft/Power-Fx/tree/main/src/libraries/Microsoft.PowerFx.Interpreter/Functions/Mutation ) should be shared with PA. Especially valuable to share CheckType implementations and ensure we have same semantics.

  1. Bonus if we can share the corresponding .txt tests.
  2. We still should require a host explicitly enables them. No change to hosting.
  3. This may require some refactoring in PA to decouple and make more shareable.
  4. Could leverage techniques added in recent RegEx PR (#1442) for associating a TexlFunction with an implementation,
anderson-joyle commented 1 year ago

Related: