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.

Sign up for GitHub

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

Jump to bottom

fix(ingest)- airflow: update subdag check for compatibility with older Airflow versions #5523

Merged
merged 5 commits into from
Jul 29, 2022
Merged

fix(ingest)- airflow: update subdag check for compatibility with older Airflow versions #5523

merged 5 commits into from
Jul 29, 2022

Commits on Jul 28, 2022

  1. use getattr to default None if no subdag

    @vcs9
    vcs9 committed Jul 28, 2022
    Configuration menu
    Copy the full SHA
    eb62deb View commit details
    Browse the repository at this point in the history

  2. add None check

    @vcs9
    vcs9 authored Jul 28, 2022
    Configuration menu
    Copy the full SHA
    76dd12c View commit details
    Browse the repository at this point in the history

  3. add other None check

    @vcs9
    vcs9 authored Jul 28, 2022
    Configuration menu
    Copy the full SHA
    77bc90d View commit details
    Browse the repository at this point in the history

Commits on Jul 29, 2022

  1. Apply suggestions from code review- double quotes 

    Co-authored-by: Harshal Sheth <[email protected]>
    @vcs9 @hsheth2
    vcs9 and hsheth2 authored Jul 29, 2022
    Configuration menu
    Copy the full SHA
    fc621f3 View commit details
    Browse the repository at this point in the history

  2. minor tweak to fix lint

    @hsheth2
    hsheth2 committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    b852834 View commit details
    Browse the repository at this point in the history