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

[dagster-tableau] Use get_asset_spec().key in DagsterTableauTranslator #26027

Merged
merged 4 commits into from
Nov 25, 2024
Merged

[dagster-tableau] Use get_asset_spec().key in DagsterTableauTranslator #26027

merged 4 commits into from
Nov 25, 2024
+42 −38

Commits on Nov 25, 2024

  1. [dagster-tableau] Use get_asset_spec().key in DagsterTableauTranslator

    @maximearmstrong
    maximearmstrong committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    fc136d2 View commit details
    Browse the repository at this point in the history

  2. Update reconstruction test

    @maximearmstrong
    maximearmstrong committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    449932e View commit details
    Browse the repository at this point in the history

  3. Update get_asset_key implementation

    @maximearmstrong
    maximearmstrong committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    666fe58 View commit details
    Browse the repository at this point in the history

  4. Use AssetSpec.replace_attributes

    @maximearmstrong
    maximearmstrong committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    3e70494 View commit details
    Browse the repository at this point in the history