barrysteyn commented on PR #28829:
URL: https://github.com/apache/airflow/pull/28829#issuecomment-1705501321

   So, this is the regex replace for this PR:
   `p = re.sub(r"(\w)\.", r"\1\..", p)`
   
   I am no expert, but it seems to me that it replaces `word.` with `word\..` - 
I think the idea is to match `.`, however all that does is ensure that there 
must be a `.`  followed by any character... That is why things are failing...
   
   I don't advocate for changing anything, but if you really want to match just 
`.`, then the regex should be  `p = re.sub(r"(\w)\.", r"\1[.]", p)`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to