eumiro commented on code in PR #33130:
URL: https://github.com/apache/airflow/pull/33130#discussion_r1288996409


##########
airflow/configuration.py:
##########
@@ -1336,7 +1325,7 @@ def getsection(self, section: str) -> 
ConfigOptionsDictType | None:
             _section.update(OrderedDict(self.items(section)))
 
         section_prefix = self._env_var_name(section, "")
-        for env_var in sorted(os.environ.keys()):
+        for env_var in sorted(os.environ):

Review Comment:
   You're right, this is not a very useful change in most cases except for a 
few characters of code. I do not have a strong opinion on it and therefore will 
avoid the change in new PRs. 



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