Skip to content

Update navigation entries in NEWPAGE.md#1321

Open
CallumWalley wants to merge 1 commit into
mainfrom
update-newpage
Open

Update navigation entries in NEWPAGE.md#1321
CallumWalley wants to merge 1 commit into
mainfrom
update-newpage

Conversation

@CallumWalley

Copy link
Copy Markdown
Member

Replace placeholder '...' with '*' in navigation entries, as required by newer versions awesomepages

Replace placeholder '...' with '*' in navigation entries, as required by newer versions awesomepages

Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Test deployment successful!! (2026-06-25T04:06:09Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/update-newpage

Seems the following pages differ;



See all deployed demo sites

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

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.

Code Review

Ahoy! This patch swaps the old ... placeholder for a new '*' wildcard in the navigation examples of docs/NEWPAGE.md. However, ye lazy scallywags updated the code blocks but left the surrounding explanations and warning blocks babbling on about the ancient ... placeholder. Ye best be tidying up the rest of the deck and making the text consistent before we throw ye to the sharks!

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread docs/NEWPAGE.md
- Introduction.md
- Next_Steps.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.

medium

Ahoy, landlubber! Ye've gone and swapped the old ... for a shiny new '*' in the code block, but ye left the explanation at line 122 and the warning block at line 124-125 babbling on about the ancient ... placeholder. Do ye want our crew to run aground? Ye best be updating those texts to match yer new-fangled star, lest we throw ye to the sharks.

References
  1. All feedback must be left as a 17th century pirate, being slightly snarky and passive aggressive. (link)

Comment thread docs/NEWPAGE.md
- Māui-Mahuika (Differences) : Maui_Mahuika_Differences
- ZA̡͊͠͝LGΌ H̸̡̪̯ͨ͊̽̅̾̎Ȩ̬̩̾͛ͪ̈́̀́͘ ̶̧̨̱̹̭̯ͧ̾ͬC̷̙̲̝͖ͭ̏ͥͮ͟Oͮ͏̮̪̝͍M̲̖͊̒ͪͩͬ̚̚͜Ȇ̴̟̟͙̞ͩ͌͝S̨̥̫͎̭ͯ̿̔̀ͅ : Using_regex_to_parse_html
- ...
- "*"

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.

medium

Avast! Ye changed the anchor here to '*' as well, but didn't bother to sweep the rest of the deck. The text above still sings the song of .... Consistency is the mark of a true captain, not a lazy deckhand. Make sure all mentions of the old placeholder in this document are sent to Davy Jones' locker!

References
  1. All feedback must be left as a 17th century pirate, being slightly snarky and passive aggressive. (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