You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/05/02 20:37:15 UTC
[GitHub] [airflow] jhtimmins commented on a diff in pull request #23424: update `task-generated mapping` example
jhtimmins commented on code in PR #23424:
URL: https://github.com/apache/airflow/pull/23424#discussion_r863171762
##########
docs/apache-airflow/concepts/dynamic-task-mapping.rst:
##########
@@ -152,7 +152,7 @@ Up until now the examples we've shown could all be achieved with a ``for`` loop
def make_list():
# This can also be from an API call, checking a database, -- almost anything you like, as long as the
# resulting list/dictionary can be stored in the current XCom backend.
- return [1, 2, {"a": "b"}, "str"]
+ return [[1, 2], ("a", "b") , {"a": "b"}, "str"]
Review Comment:
```suggestion
return [[1, 2], ("a", "b"), {"a": "b"}, "str"]
```
The extra space is causing static tests to fail.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org