-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[BUGFIX] row_conditions
now allow whitespace
#7313
[BUGFIX] row_conditions
now allow whitespace
#7313
Conversation
✅ Deploy Preview for niobium-lead-7998 canceled.
|
column_name = Combine( | ||
Suppress(Literal('col("')) | ||
+ Word(alphas, f"{alphanums}_.").setResultsName("column") | ||
+ Word(alphas, f"{alphanums}_-.").setResultsName("column") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
allows for -
to be used in column names, along with _
@@ -56,7 +57,7 @@ def _set_notnull(s, l, t) -> None: # noqa: E741 # ambiguous name `l` | |||
ops = (gt ^ lt ^ ge ^ le ^ eq ^ ne).setResultsName("op") | |||
fnumber = Regex(r"[+-]?\d+(?:\.\d*)?(?:[eE][+-]?\d+)?").setResultsName("fnumber") | |||
punctuation_without_apostrophe = punctuation.replace('"', "").replace("'", "") | |||
condition_value_chars = alphanums + punctuation_without_apostrophe | |||
condition_value_chars = alphanums + punctuation_without_apostrophe + SPACE_CHARS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adds SPACE_CHARS
to valid condition_value_chars
, which is turned into the condition value
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
@@ -42,9 +42,10 @@ def _set_notnull(s, l, t) -> None: # noqa: E741 # ambiguous name `l` | |||
t["notnull"] = True | |||
|
|||
|
|||
SPACE_CHARS = " " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add \t
condition_value_chars
inrow_conditions
._
and-
can both appear in column names forrow_conditions
.Definition of Done