microsoft / microsoft/Power-Fx

Regular expression MatchOptions and Match enums not added with Config.EnableRegExFunctions()

Open
#1,664 0 comments 0 reactions 1 assignee View on GitHub

@LucGenetier is already working on this.

Since Jul 14, 2023.

Regular Expressions REPL
Dominant language
C#
Stars
3.4k
Forks
358
Avg merge
10h 34m
Merged PRs (30d)
3

Description

In the REPL, adding a call to config.EnableRegExFunctions around line 100 (near config.EnableSetFunction and config.EnableParseJSONFunction) adds the IsMatch, Match, and MatchAll functions as it should, but it does not add the associated enums Match and MatchOptions. No further actions should be necessary to get these enums into the namespace.

Debugging, something similar to internal EnumStoreBuilder WithRequiredEnums(TexlFunctionSet functions) on these functions needs to be called in EnumStoreBuilder.cs when these functions are enabled.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.