-
Notifications
You must be signed in to change notification settings - Fork 0
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
Ideas: possibly improvements to the output of the metrics command #81
Comments
Regex switch
The relevant arguments are:
Notes
Case insensitiveThere should be a switch The default value should be |
sgreenbury
added a commit
that referenced
this issue
Sep 6, 2024
Adds SearchConfig with `MatchType` (Exact, Regex, Contains, Startswith), and `CaseSensitivity` (Insensitive or Sensitive), adds a test, and incorporates into CLI with defaults for: - `SearchText` when across all search contexts (regex) - `MetricsId` (startswith, case insensitive)
sgreenbury
added a commit
that referenced
this issue
Sep 10, 2024
Adds SearchConfig with `MatchType` (Exact, Regex, Contains, Startswith), and `CaseSensitivity` (Insensitive or Sensitive), adds a test, and incorporates into CLI with defaults for: - `SearchText` when across all search contexts (regex) - `MetricsId` (startswith, case insensitive)
sgreenbury
added a commit
that referenced
this issue
Sep 10, 2024
Adds SearchConfig with `MatchType` (Exact, Regex, Contains, Startswith), and `CaseSensitivity` (Insensitive or Sensitive), adds a test, and incorporates into CLI with defaults for: - `SearchText` when across all search contexts (regex) - `MetricsId` (startswith, case insensitive)
33 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Draft:
source_data_url
match_type
: "exact", "regex" to include in CLI but also add "startswith", "contains" internal API)The text was updated successfully, but these errors were encountered: