minor changes to documentations #1243
Conversation
lamtnguyen989
requested review from
a team,
bmiddha and
clee231
as code owners
| .env | ||
|
|
||
| # You know it's coming | ||
| docs/Infrastructure/NERV/ |
There was a problem hiding this comment.
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"
There was a problem hiding this comment.
In that case, don't stage them for commit or stash them before staging or use stacked branches.
There was a problem hiding this comment.
Okay I'll look into it tomorrow, still kinda of a git noob
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-sea-0f799c410-1243.centralus.azurestaticapps.net |
There was a problem hiding this comment.
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.mdto 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.mdby combining two rows into a single entry for patch panels - Added a new directory to
.gitignorefor 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.
| ::: | ||
|
|
||
| The following pages already contain some information. | ||
| - [**Infrastructure**](Infrastructure/Overview.md): which are holding the current information about the organization's servers |
There was a problem hiding this comment.
Grammatical error: "which are holding" should be "which holds" or "which hold" (without "are"). The phrase should match subject-verb agreement.
| - [**Infrastructure**](Infrastructure/Overview.md): which are holding the current information about the organization's servers | |
| - [**Infrastructure**](Infrastructure/Overview.md): which holds the current information about the organization's servers |
Description
Basically some minor stylistic changes to
docs/directory documentations.Also update the
introduction.mdfile at the head ofdocs/Issues
Checklist