dagster icon indicating copy to clipboard operation
dagster copied to clipboard

[du] updating based on API changes from 1dot7

Open tacastillo opened this issue 9 months ago • 5 comments

Summary & Motivation

based on the API changes listed here https://www.notion.so/dagster/Proposal-of-DU-updates-for-1-7-2bd3ec68fc8b41dfa90099ade8d71a96

How I Tested These Changes

tacastillo avatar May 08 '24 17:05 tacastillo

  • #21726 Graphite
  • #21725 Graphite 👈
  • master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tacastillo and the rest of your teammates on Graphite Graphite

tacastillo avatar May 08 '24 17:05 tacastillo

Deploy preview for dagster-docs ready!

Preview available at https://dagster-docs-afaw88qm8-elementl.vercel.app https://05-08--du-updating-based-on-api-changes-from-1dot7.dagster.dagster-docs.io

Direct link to changed pages:

github-actions[bot] avatar May 08 '24 17:05 github-actions[bot]

Corresponding project pull request for tracking - https://github.com/dagster-io/project-dagster-university/pull/25

cmpadden avatar May 08 '24 17:05 cmpadden

Yup, @tacastillo just one tiny comment on if _ typehinting_ is one or two words, other than that lgtm.

cmpadden avatar May 09 '24 21:05 cmpadden

Deploy preview for dagster-university ready!

✅ Preview https://dagster-university-f2k7atzbe-elementl.vercel.app https://05-08--du-updating-based-on-api-changes-from-1dot7.dagster-university.dagster-docs.io

Built with commit 73e96b1def5f650c758d7719958ec0a6e19ae6ec. This pull request is being automatically deployed with vercel-action

github-actions[bot] avatar May 10 '24 13:05 github-actions[bot]

Merge activity

  • May 13, 1:34 PM EDT: @tacastillo started a stack merge that includes this pull request via Graphite.
  • May 13, 1:36 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 13, 1:37 PM EDT: @tacastillo merged this pull request with Graphite.

tacastillo avatar May 13 '24 17:05 tacastillo