Page MenuHomeElementl

[Memoization 10/13] Add top level exports for memoization + API reference page
ClosedPublic

Authored by cdecarolis on Aug 9 2021, 10:58 PM.

Details

Summary

Title. These are the things necessary to export at the top level in the new memoization landscape.

Test Plan

unit

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 9 2021, 11:42 PM
Harbormaster failed remote builds in B34634: Diff 42845!

Do these have accompanying entries in the API reference?

Add an API reference page for memoization

cdecarolis retitled this revision from [Memoization 10/n] Add top level exports for memoization to [Memoization 10/n] Add top level exports for memoization + API reference page.Aug 10 2021, 8:24 PM
cdecarolis retitled this revision from [Memoization 10/n] Add top level exports for memoization + API reference page to [Memoization 10/12] Add top level exports for memoization + API reference page.Aug 10 2021, 8:37 PM
cdecarolis retitled this revision from [Memoization 10/12] Add top level exports for memoization + API reference page to [Memoization 10/13] Add top level exports for memoization + API reference page.Aug 11 2021, 12:21 AM
sandyryza added inline comments.
docs/sphinx/sections/api/apidocs/memoization.rst
5

Comma unnecessary here, because the latter isn't an independency clause.

This revision is now accepted and ready to land.Aug 12 2021, 3:14 PM