Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detect unmatched skip begin/end commands #1309

Merged
merged 4 commits into from
Oct 18, 2024
Merged

Conversation

mikekgfb
Copy link
Contributor

warn about unmatched skip markup in docs run thru updown processor (processing test markup for markdown files)

cc: @swolchok

The updown processor should warn when skip begin/end are garbled
adjust nesting level on skip end
Copy link

pytorch-bot bot commented Oct 16, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/torchchat/1309

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

✅ No Failures

As of commit 7982fcd with merge base 791fe82 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 16, 2024
@malfet malfet closed this Oct 16, 2024
@malfet malfet reopened this Oct 16, 2024
@malfet
Copy link
Contributor

malfet commented Oct 16, 2024

Sorry for closing, I've assumed it's an old PR as it showed a lots of conflicts. Reopening...

replacing llama3 with stories15 meant we got stories15.1.
Fixing for `run-docs readme`
@mikekgfb mikekgfb changed the title Patch 5 Detect unmatched skip begin/end commands Oct 16, 2024
@mikekgfb
Copy link
Contributor Author

Sorry for closing, I've assumed it's an old PR as it showed a lots of conflicts. Reopening...

No worries. Sync'ed and added a fix for llama3 -> llama3.1 (not sure how this has been working, local runs created stories15.1 after substituting s/llama3/stories15/

@mikekgfb
Copy link
Contributor Author

@malfet please have a look at this test https://github.com/pytorch/torchchat/actions/runs/11375390708/job/31652543330?pr=1309 - it appears to be failing, yet is indicated as passing (independent of this PR)

Ditto https://github.com/pytorch/torchchat/actions/runs/11375390717/job/31652547554?pr=1309 which is explicitly erroring out with exit 1, yet also shown as passing?

@Jack-Khuu Jack-Khuu merged commit 4f2f4fb into pytorch:main Oct 18, 2024
52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants