shivaam commented on PR #63657: URL: https://github.com/apache/airflow/pull/63657#issuecomment-4499638196
> LGTM. That's an ugly docstring, and maybe it would look nicer as something like > > ``` > def _process_workloads(self, workload_items: Sequence[workloads.All]) -> None: > """ > Process queued workloads by dispatching to ECS. > > :sphinx-autoapi-skip: > """ > ``` > > but it _was_ my suggestion, it works, and I'm not generally keen on docstrings that just rephrase the method name. Approving. Let me know if there are any other comments or if this ready to be merged. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
