Skip to content

Conversation

@SamErde
Copy link
Contributor

@SamErde SamErde commented Dec 15, 2025

Description

Update the docs web site navigation sidebar:

  • Re-order to follow usage flow and group test-related pages
  • Add emoji to some page titles for visual consistency
  • Update some formatting for consistency

Contribution Checklist

Before submitting this PR, please confirm you have completed the following:

  • 📖 Read the guidelines for contributing to this repository.
  • 🧪 Ensure the build and unit tests pass by running /powershell/tests/pester.ps1 on your local system.

 

Join us at the Maester repository discussions 💬 or Entra Discord 🧑‍💻 for more help and conversations!

Copilot AI review requested due to automatic review settings December 15, 2025 11:27
@SamErde SamErde requested a review from a team as a code owner December 15, 2025 11:27
@SamErde SamErde added the documentation Improvements or additions to documentation label Dec 15, 2025
Copy link
Contributor

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 reorganizes the documentation website's navigation sidebar to improve usability by following a more logical usage flow. The changes group test-related pages together, add emojis to page titles for visual consistency, and update formatting across several documentation files.

  • Restructured sidebar navigation to follow usage flow (configure → connect → export → monitor → alert)
  • Added emojis to page titles for visual hierarchy and consistency
  • Moved test-related items (ca-what-if, updating-tests, writing custom tests) under the "Tests Overview" section

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
website/sidebars.js Reordered navigation items, grouped test-related pages under Tests Overview section, and added visual dividers
website/docs/updating-tests.md Added emoji to title and adjusted heading levels for consistency
website/docs/intro.md Changed title to "📦 Installation guide" to better reflect content
website/docs/export-results.md Added emoji to title and removed trailing whitespace
website/docs/ca-what-if.md Updated title capitalization and improved code formatting consistency

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

SamErde and others added 3 commits December 15, 2025 08:27
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.


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

@SamErde SamErde marked this pull request as draft January 21, 2026 12:53
@SamErde SamErde marked this pull request as ready for review January 21, 2026 12:54
@SamErde SamErde requested a review from Copilot January 21, 2026 12:56
@SamErde SamErde self-assigned this Jan 21, 2026
Removed commented-out installation entry from sidebars.

```
siteSidebar: [
  "intro",
  //installation
```
@SamErde SamErde merged commit 73eff98 into maester365:main Jan 21, 2026
3 checks passed
@SamErde SamErde deleted the docs-NavigationSidebar branch January 21, 2026 12:58
Copy link
Contributor

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

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


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

collapsed: true,
collapsible: true,
items: [{ type: "autogenerated", dirName: "writing-tests" }],
}
Copy link

Copilot AI Jan 21, 2026

Choose a reason for hiding this comment

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

Missing comma after the closing brace of the "Writing custom tests" category object. This will cause a JavaScript syntax error when the sidebar configuration is loaded. A comma is required to separate array elements.

Suggested change
}
},

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant