Skip to content

large file suggestion#1245

Merged
geoffreyweal merged 1 commit intomainfrom
data_transfer
May 4, 2026
Merged

large file suggestion#1245
geoffreyweal merged 1 commit intomainfrom
data_transfer

Conversation

@AgentKumara
Copy link
Copy Markdown
Collaborator

No description provided.

@AgentKumara AgentKumara requested a review from geoffreyweal May 3, 2026 23:31
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 3, 2026

Test deployment successful!! (2026-05-03T23:31:22Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/data_transfer

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

Avast! This patch swaps out the old RClone talk for OnDemand and adds a warning for those fools trying to move massive booty through a browser. One of the crew pointed out that labeling an info block as a "Note" is as useless as a map with no "X", violating the style guide's ban on non-descriptive titles. Fix the title or prepare to be marooned.



Mahuika OnDemand only a web browser to use. The instructions to use it are the same whether your are connecting from a Windows, Mac or a Linux computer.
!!! info "Note"
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

Arr, I see ye've added a 'Note' title to yer info block. How very helpful of ye to tell us it's a note, as if the big blue box weren't enough of a clue. The Repository Style Guide (line 281) says to drop the non-descriptive titles. I'd tell ye to walk the plank, but I'll settle for ye fixin' the code instead, ye lazy scallywag.

Suggested change
!!! info "Note"
!!! info
References
  1. Refrain from using unnecessary or non-descriptive titles for admonitions; better to leave them titleless. (link)

@geoffreyweal geoffreyweal merged commit 862b019 into main May 4, 2026
8 checks passed
@geoffreyweal geoffreyweal deleted the data_transfer branch May 4, 2026 03:08
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.

2 participants