Skip to content

include auto-docstring check in the modular ci.#13004

Merged
yiyixuxu merged 1 commit intomodular-doc-improvfrom
include-doc-check-in-ci
Jan 24, 2026
Merged

include auto-docstring check in the modular ci.#13004
yiyixuxu merged 1 commit intomodular-doc-improvfrom
include-doc-check-in-ci

Conversation

@sayakpaul
Copy link
Member

What does this PR do?

Just a small CI nit on top of #12958

@sayakpaul sayakpaul requested a review from yiyixuxu January 20, 2026 10:13
Copy link
Collaborator

@yiyixuxu yiyixuxu left a comment

Choose a reason for hiding this comment

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

thanks

@yiyixuxu yiyixuxu merged commit 412e51c into modular-doc-improv Jan 24, 2026
1 check passed
@yiyixuxu yiyixuxu deleted the include-doc-check-in-ci branch January 24, 2026 08:34
yiyixuxu added a commit that referenced this pull request Jan 27, 2026
* up

* up up

* update outputs

* style

* add modular_auto_docstring!

* more auto docstring

* style

* up up up

* more more

* up

* address feedbacks

* add TODO in the description for empty docstring

* refactor based on dhruv's feedback: remove the class method

* add template method

* up

* up up up

* apply auto docstring

* make style

* rmove space in make docstring

* Apply suggestions from code review

* revert change in z

* fix

* Apply style fixes

* include auto-docstring check in the modular ci. (#13004)

* Run ruff format after auto docstring generation

* up

* upup

* upup

* style

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Sayak Paul <spsayakpaul@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants