- Notifications
You must be signed in to change notification settings - Fork11.2k
Docs: Cleaned and added proper JOBDIR file documentation (fixes #4842)#7110
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Gallaecio commentedOct 20, 2025
Please review your own changes :) |
veerasai06 commentedOct 20, 2025
Hi! I’ve reviewed all changes and double-checked the documentation and formatting. Ready for review. |
Gallaecio commentedOct 20, 2025
Can you tell me what’s the first line that your changes add? |
veerasai06 commentedOct 21, 2025
The first line added by my changes is Files Stored in JOBDIR, which introduces the new section explaining the files stored in the JOBDIR directory. |
Gallaecio commentedOct 21, 2025
It is not. You are adding another line earlier. |
veerasai06 commentedOct 22, 2025
Ah, you’re right — I see now that the first line I added is actually nano docs/topics/jobs.rst, which was added by mistake. I’ll remove that line from the commit and update the PR. Thanks for catching that! |
Uh oh!
There was an error while loading.Please reload this page.
Fixes#4842.