Skip to content

Comments

Set up claude#3779

Open
sekyondaMeta wants to merge 2 commits intomainfrom
claudeSetUp
Open

Set up claude#3779
sekyondaMeta wants to merge 2 commits intomainfrom
claudeSetUp

Conversation

@sekyondaMeta
Copy link
Contributor

Set up claude in tutorials repo

Set up claude in tutorials repo
@pytorch-bot
Copy link

pytorch-bot bot commented Feb 19, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/3779

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 2c57500 with merge base 6f800e1 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

This repo uses `lintrunner`. Do not use `spin`, `flake8`, or other linters directly.

```
pip install lintrunner==0.12.5

Choose a reason for hiding this comment

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

wouldn't it make more sense to pre-install these tools in the workflow?

3. Add the tutorial to the corresponding `toctree` in `index.rst`.
4. Add a square, high-resolution thumbnail image to `_static/img/thumbnails/cropped/`.

# Commit Messages

Choose a reason for hiding this comment

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

I'm not sure the current permissions allow committing:

    permissions:
      contents: read

also, please note that write access could be a security concern.

@@ -0,0 +1,82 @@
# Environment

If any tool you're trying to use (make, sphinx-build, lintrunner, python, etc.) is missing, always stop and ask the user if an environment is needed. Do NOT try to find alternatives or install these tools.

Choose a reason for hiding this comment

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

this conflicts with the installation instructions below. Ask claude to analyze the text for contradictions locally.

Copy link

@izaitsevfb izaitsevfb left a comment

Choose a reason for hiding this comment

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

stamping to unblock, but please see the comments

@@ -0,0 +1,82 @@
# Environment

Choose a reason for hiding this comment

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

oh, another note:

CLAUDE.md contains general guidelines for agents (including those run by people who check out the repo locally)

you might want to extract review-specific guidelines into a separate skill

Co-authored-by: Ivan Zaitsev <108101595+izaitsevfb@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants