[ https://issues.apache.org/jira/browse/AIRFLOW-5487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16930154#comment-16930154 ]
ASF subversion and git services commented on AIRFLOW-5487: ---------------------------------------------------------- Commit b2e3e8c0718142d4cb0387f46cd77c15b67cc1e9 in airflow's branch refs/heads/test-gitlab-ci from Adan Rosales [ https://gitbox.apache.org/repos/asf?p=airflow.git;h=b2e3e8c ] [AIRFLOW-5487]Fix unused warning var (#6111) > airflow/utils/strings.py: Fix unused warning variable > ----------------------------------------------------- > > Key: AIRFLOW-5487 > URL: https://issues.apache.org/jira/browse/AIRFLOW-5487 > Project: Apache Airflow > Issue Type: Bug > Components: utils > Affects Versions: 1.10.6 > Reporter: Jakob Homan > Priority: Minor > Labels: ccoss2019, newbie > > Note: This ticket's being created to facilitate a new contributor's workshop > for Airflow. After the workshop has completed, I'll mark these all available > for anyone that might like to take them on. > airflow/utils/strings.py:27 > {code:java} > def get_random_string(length=8, choices=string.ascii_letters + string.digits): > ''' > Generate random string > ''' > return ''.join([choice(choices) for i in range(length)]) {code} > Here we use an `i` for a placeholder variable, but this gets flagge as an > unused variable. We can replace the `i` with `_` and avoid this warning. -- This message was sent by Atlassian Jira (v8.3.2#803003)