[GitHub] [airflow] dstandish commented on pull request #31754: Control permissibility of driver config in extra from airflow.cfg

["dstandish (via GitHub)" <gi...@apache.org>]

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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org