Skip to content

Commit

Permalink
Add Map to config type API docs (#7154)
Browse files Browse the repository at this point in the history
Summary:
This is referenced in https://docs.dagster.io/_apidocs/config#config-types but there is no link.

Test Plan: Load api docs locally
  • Loading branch information
gibsondan committed Mar 22, 2022
1 parent f426230 commit 01294b4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions docs/sphinx/sections/api/apidocs/config.rst
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ builtin types above.

.. autoclass:: Shape

.. autoclass:: Map

.. autoclass:: Array

.. autoclass:: Noneable
Expand Down
2 changes: 1 addition & 1 deletion python_modules/dagster/dagster/config/field_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ class Permissive(_ConfigHasFields):
.. code-block:: python
@op(config_schema=Field(Permissive({'required': Field(String)})))
def partially_specified_config(context) -> List:
def map_config_op(context) -> List:
return sorted(list(context.op_config.items()))
"""

Expand Down

1 comment on commit 01294b4

@vercel
Copy link

@vercel vercel bot commented on 01294b4 Mar 22, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.