Page MenuHomeElementl

[crag] api docs section
ClosedPublic

Authored by alangenfeld on Wed, Jul 7, 9:51 PM.

Details

Test Plan

docs/ make dev

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

any and all copy edits / suggestions welcome - especially *

docs/sphinx/sections/api/apidocs/experimental.rst
5–50

*

I think this looks great

This revision is now accepted and ready to land.Thu, Jul 8, 1:40 AM

I'm on board, a few changes I would make to some of the docs sections.

docs/sphinx/sections/api/apidocs/experimental.rst
11

I think this is confusingly worded, maybe
"Graph is the replacement for x. It has a more intuitive name, and ...

The second bit is kinda confusing but I'm not sure how to make it better. Maybe just say "defines a dependency graph that can be the top-level for a dagster execution, or easily composed into other executions."

python_modules/dagster/dagster/core/definitions/pipeline.py
532–535

I don't think this actually comes off as confusing as I thought it would.

This revision was landed with ongoing or failed builds.Thu, Jul 8, 3:04 PM
This revision was automatically updated to reflect the committed changes.