Conversation
Set up claude in tutorials repo
🔗 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 FailuresAs of commit 2c57500 with merge base 6f800e1 ( 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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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. | |||
There was a problem hiding this comment.
this conflicts with the installation instructions below. Ask claude to analyze the text for contradictions locally.
izaitsevfb
left a comment
There was a problem hiding this comment.
stamping to unblock, but please see the comments
| @@ -0,0 +1,82 @@ | |||
| # Environment | |||
There was a problem hiding this comment.
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>
Set up claude in tutorials repo