Skip to content

minor changes to documentations#1243

Draft
lamtnguyen989 wants to merge 4 commits intomainfrom
minor-docs-changes
Draft

minor changes to documentations#1243
lamtnguyen989 wants to merge 4 commits intomainfrom
minor-docs-changes

Conversation

@lamtnguyen989
Copy link
Member

Description

Basically some minor stylistic changes to docs/ directory documentations.

Also update the introduction.md file at the head of docs/

Issues

Checklist

  • I've labeled the PR appropriately.
  • I've have built the website and verified that my changes work.
  • I've linked the appropriate issues and related PRs.

.env

# You know it's coming
docs/Infrastructure/NERV/ No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

why gitignore this??

Copy link
Member Author

Choose a reason for hiding this comment

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

Just simply made a few a .md files. I have not collect enough details to make a full documentations yet and I don't want to add "bloat"

Copy link
Member

Choose a reason for hiding this comment

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

In that case, don't stage them for commit or stash them before staging or use stacked branches.

Copy link
Member Author

Choose a reason for hiding this comment

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

Okay I'll look into it tomorrow, still kinda of a git noob

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-sea-0f799c410-1243.centralus.azurestaticapps.net

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR makes minor stylistic improvements to documentation files, updating the introduction page and consolidating table entries in the infrastructure documentation.

  • Updated introduction.md to replace the outdated "migration in process" caution with a clearer description of the two main documentation types
  • Consolidated rack layout table entries in Infrastructure/Overview.md by combining two rows into a single entry for patch panels
  • Added a new directory to .gitignore for future NERV infrastructure documentation

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
docs/introduction.md Replaced outdated migration notice with structured documentation overview and links
docs/Infrastructure/Overview.md Combined two separate patch panel rows (41 and 40) into a single consolidated entry
.gitignore Added exclusion for future NERV infrastructure documentation directory

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-sea-0f799c410-1243.centralus.azurestaticapps.net

@lamtnguyen989 lamtnguyen989 marked this pull request as draft February 13, 2026 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants