Skip to content

some new commits cause redirect errors. Fixed this batch but should look into it more closely#1213

Merged
WesHarrell merged 1 commit intomainfrom
fix-more-redirects
Mar 27, 2026
Merged

some new commits cause redirect errors. Fixed this batch but should look into it more closely#1213
WesHarrell merged 1 commit intomainfrom
fix-more-redirects

Conversation

@WesHarrell
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

Test deployment successful!! (2026-03-27T01:32:50Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/fix-more-redirects

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

Ye've seen fit to trim the redirect map, shortening one path while tossing the Machine Learning FAQ entry into the briny deep. While the shorter name might please the eyes, deleting a redirect instead of updating it is a fine way to run our users aground on a 404 reef. If ye've renamed the file to fit the 24-character rule, have the decency to update the map so the poor souls can actually find their treasure.

General/FAQs/What_is_a_core_file.md : Getting_Started/FAQs/What_is_a_core_file.md
General/FAQs/What_Is_A_Trusted_Device.md : Getting_Started/FAQs/What_Is_A_Trusted_Device.md
General/FAQs/What_is_Multiple_Factor_Authentication_MFA.md : Getting_Started/FAQs/What_is_Multiple_Factor_Authentication_MFA.md
General/FAQs/What_software_environments_are_optimised_for_Machine_Learning_and_data_science.md : Getting_Started/FAQs/What_software_environments_are_optimised_for_Machine_Learning_and_data_science.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! It seems this old map entry has been cast into the sea. While the original filename was as long as a kraken's tentacle, simply scuttling the redirect might lead poor sailors to a dreaded 404 page if the destination was merely renamed.

If the page still exists under a shorter, more shipshape name, ye should be updating this redirect, not deleting it. A proper captain ensures all paths lead to treasure, not to ruin.

References
  1. The style guide advises keeping filenames short to prevent wrapping in the navigation bar (around 24 characters). The filename in this removed redirect is excessively long, suggesting it may have been renamed. (link)

@WesHarrell WesHarrell merged commit c328d2d into main Mar 27, 2026
8 checks passed
@WesHarrell WesHarrell deleted the fix-more-redirects branch March 27, 2026 01:34
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