jscheffl commented on code in PR #45162:
URL: https://github.com/apache/airflow/pull/45162#discussion_r1895049708


##########
.github/workflows/ci.yml:
##########
@@ -21,9 +21,15 @@ on:  # yamllint disable-line rule:truthy
   schedule:
     - cron: '28 1,7,13,19 * * *'
   push:
-    branches: ['v[0-9]+-[0-9]+-test', 'providers-*/v[0-9]+-[0-9]+]']
+    branches:

Review Comment:
   YAML is made to be less boilerplate. No quotes needed except if you want to 
force bool/int to be string. I favor w/o quotes. But might be a matter of 
taste. No difference in security/safety. All YAML parsers work without. Else it 
is a bad YAML parser.
   
   --> 
https://stackoverflow.com/questions/19109912/yaml-do-i-need-quotes-for-strings-in-yaml
 ?



-- 
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