Skip to content

Adding Section about running jobs from OnDemand#1235

Draft
geoffreyweal wants to merge 9 commits intomainfrom
running-jobs-from-ondemand
Draft

Adding Section about running jobs from OnDemand#1235
geoffreyweal wants to merge 9 commits intomainfrom
running-jobs-from-ondemand

Conversation

@geoffreyweal
Copy link
Copy Markdown
Contributor

Adding Section about running jobs from OnDemand

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026

Test deployment successful!! (2026-04-22T20:33:58Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/running-jobs-from-ondemand

Seems the following pages differ;



See all deployed demo sites

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a navigation configuration for OnDemand jobs and a redirect for the MATLAB documentation. It seems ye think the rules of the sea don't apply to ye, as ye've named articles in the navigation file instead of using page metadata and used a '*' instead of the proper '...' to include other pages. Correct these amateur mistakes before the Captain sees 'em.

@@ -0,0 +1,4 @@
---
nav:
- Running Jobs from OnDemand: running_jobs_from_ondemand.md
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

low

Ahoy! I see ye've decided the rules don't apply to a 'special' sailor like yourself. Naming articles in the nav file is a fine way to show everyone ye can't be bothered to read the metadata instructions. Use the title parameter in the page itself, ye lazy scallywag.

  - running_jobs_from_ondemand.md
References
  1. Don't use .page.yml to set the names of articles or category-articles; use the title parameter in the page's metadata instead. (link)

---
nav:
- Running Jobs from OnDemand: running_jobs_from_ondemand.md
- "*"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

low

Avast! A * instead of ...? Truly, yer creativity knows no bounds. It's a shame the plugin doesn't share yer imagination and might just hide yer pages in the locker instead. Stick to the map and use ... like the rest of the crew.

  - ...
References
  1. Use '...' in .page.yml to include all other pages in default order; otherwise, some pages may not be rendered. (link)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant