mohamedawnallah commented on code in PR #35733: URL: https://github.com/apache/beam/pull/35733#discussion_r2241457819
########## sdks/python/scripts/run_pytest.sh: ########## @@ -41,17 +48,29 @@ # Initialize the user_marker variable. user_marker="" -# Isolate the user-provided -m argument (matching only). -if [[ $pytest_args =~ "-m" ]]; then - # Extract the marker value using the defined regex. - user_marker=$(echo "$pytest_args" | sed -nE "s/.*$marker_regex.*/\1/p") -fi +# Define regex pattern for quoted strings +quotes_regex="^[\"\'](.*)[\"\']$" Review Comment: > it can incorrectly match strings with mixed quotes @gemini-code-assist The input supposed to be valid quotes -- 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]
