dstandish commented on PR #31754: URL: https://github.com/apache/airflow/pull/31754#issuecomment-1581150331
if you guys both feel that way after the following argument then i'll change it * i agree that double under is better from readability perspective * however, i think it's much worse from user discoverability perspective and i think that's most important * i think as a user, when you are dealing with an invalid character, most intuitive would be that you replace it with a valid one -- not that you replace it with _two_ valid ones. what say you @potiuk @uranusjr -- 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]
