Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean-ups around task-mapping code #26879

Merged
merged 1 commit into from
Oct 11, 2022

Commits on Oct 6, 2022

  1. Clean-ups around task-mapping code

    There should be no change in functionality. A few things are involved:
    
    1. The expand_mapped_literals closure is declaring a 'sequence' argument
       that is always None. Remove it.
    2. The run_time_mapped_ti_count method is never used in isolation, but
       combined with parse_time_mapped_ti_count. We should just combine the
       calls -- actually, the run-time variant already encompasses the
       parse-time variant, so the latter call can simply be removed.
    3. The TaskInstance import in _revise_mapped_task_indexes is redundant
       (the class is already imported globally) and is removed.
    4. Various typing fixups.
    uranusjr committed Oct 6, 2022
    Configuration menu
    Copy the full SHA
    38707b4 View commit details
    Browse the repository at this point in the history