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
Branch
top_level_exports_memoization
Lint
Lint Passed
Unit
No Test Coverage

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
4 ↗(On Diff #42851)

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