Skip to content

removed status of new#1276

Merged
janamakar merged 3 commits into
mainfrom
janamakar-patch-1
May 17, 2026
Merged

removed status of new#1276
janamakar merged 3 commits into
mainfrom
janamakar-patch-1

Conversation

@janamakar
Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Jana Makar <36392208+janamakar@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 17, 2026

Test deployment successful!! (2026-05-17T23:38:35Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/janamakar-patch-1

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 updates the metadata for the scratch filesystem autodeletion announcement by clearing the status field. The reviewer rightly barked that if the status ain't 'new' or 'deprecated', ye should scuttle the whole line instead of leaving a blank crate for us to trip over. Follow the code of the style guide and toss that empty line overboard, ye lazy sea dog.

Title: Autodelete of Scratch Filesystem
description: A page sharing the details of changes to scratch filesystem deletion policy
status: new
status:
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

Oh, how generous of ye to leave a blank 'status' line for the rest of us to wonder about. If it's not 'new' anymore, just toss the whole line overboard. We don't need no empty crates clutterin' the deck, ye lazy sea dog.

References
  1. The repository style guide (line 176) specifies that the 'status' parameter should be 'new' or 'deprecated'. If neither applies, the parameter should be removed entirely rather than left empty. (link)

janamakar added 2 commits May 18, 2026 11:34
Signed-off-by: Jana Makar <36392208+janamakar@users.noreply.github.com>
@janamakar janamakar merged commit 44bc1b5 into main May 17, 2026
8 checks passed
@janamakar janamakar deleted the janamakar-patch-1 branch May 17, 2026 23:38
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