Skip to content

Conversation

sekyondaMeta
Copy link
Contributor

Making the link checker Git hub action simpler while also adding ignores for 403 and 429 errors which are usually a result of a timeout but not an indicator of an issue.

Updates the action to use the checkout and changedfiles actions to check the PR files.

See screenshots of tests:

Screenshot 2024-10-23 at 09 30 27 Screenshot 2024-10-23 at 09 30 53

Checklist

  • The issue that is being fixed is referred in the description (see above "Fixes #ISSUE_NUMBER")
  • Only one issue is addressed in this pull request
  • Labels from the issue that this PR is fixing are added to this pull request
  • No unnecessary issues are included into this pull request.

@sekyondaMeta sekyondaMeta requested a review from svekars October 23, 2024 14:12
Copy link

pytorch-bot bot commented Oct 23, 2024

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit 59c9045 with merge base f2b930e (image):
💚 Looks good so far! There are no failures yet. 💚

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

@svekars svekars added the build issue Issues relating to the tutorials build label Oct 23, 2024
@sekyondaMeta sekyondaMeta merged commit 18abd43 into pytorch:main Oct 23, 2024
20 checks passed
@gukoff
Copy link

gukoff commented Feb 5, 2025

@sekyondaMeta @svekars

FYI link-checking only the changed files will now fail to detect problems with the links within the repository.

E.g. you move a file to which there is a link from an existing markdown document. The pipeline will fail to detect such a broken link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build issue Issues relating to the tutorials build cla signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants