You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Jakob Homan (Jira)" <ji...@apache.org> on 2019/09/13 22:14:00 UTC

[jira] [Created] (AIRFLOW-5487) airflow/utils/strings.py: Fix unused warning variable

Jakob Homan created AIRFLOW-5487:
------------------------------------

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


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)