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

DM-38779: Use resolved DatasetRef in ingest #452

Merged
merged 3 commits into from
May 4, 2023
Merged

DM-38779: Use resolved DatasetRef in ingest #452

merged 3 commits into from
May 4, 2023

Commits on May 4, 2023

  1. Use resolved DatasetRef in ingest

    @timj
    timj committed May 4, 2023
    Configuration menu
    Copy the full SHA
    56902d6 View commit details
    Browse the repository at this point in the history

  2. Add news fragment

    @timj
    timj committed May 4, 2023
    Configuration menu
    Copy the full SHA
    aa536fd View commit details
    Browse the repository at this point in the history

  3. Change the annotation from str | None to str for run parameter in ingest 

    It was always going to be defined but the annotation had assumed
that it might be defaulted.
    @timj
    timj committed May 4, 2023
    Configuration menu
    Copy the full SHA
    f189e8d View commit details
    Browse the repository at this point in the history