Skip to content

Conversation

@SimonWahlin
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings July 7, 2025 05:16

This comment was marked as outdated.

SimonWahlin and others added 2 commits July 7, 2025 07:17
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

Linux Test Results

 1 files  16 suites   4s ⏱️
71 tests 70 ✅ 0 💤 1 ❌
72 runs  71 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 0c7ea51.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

Win Test Results

 1 files  16 suites   4s ⏱️
71 tests 70 ✅ 0 💤 1 ❌
72 runs  71 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 0c7ea51.

♻️ This comment has been updated with latest results.

@SimonWahlin SimonWahlin requested a review from Copilot July 7, 2025 05:20
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

Enhance the README with detailed usage examples and a comparison between Search-AzResourceGraph and the existing Search-AzGraph cmdlet.

  • Added a step-by-step Usage section showing how to connect and query.
  • Introduced a new comparison section highlighting built-in paging/throttling and size/space benefits.
  • Provided side-by-side code samples for paging with Search-AzGraph vs. the simpler Search-AzResourceGraph approach.
Comments suppressed due to low confidence (2)

README.md:29

  • [nitpick] Consider rephrasing to be more concise and active, e.g. It has no dependency on the Az module, making it ideal for use in specialized containers….
It works without any dependency on the Az module, which makes it suitable for use in, for example, specialized containers where AzResourceGraph and AzAuth together use only 15 MB of disk space,

README.md:61

  • Remove the leading + inside the code block so the example error message isn’t confused with diff syntax.
"Response payload size is 16802910, and has exceeded the limit of 16777216. Please consider querying less data at a time and make paginated call if needed."

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@SimonWahlin SimonWahlin requested a review from PalmEmanuel July 7, 2025 10:12
@SimonWahlin SimonWahlin merged commit 88d166c into main Jul 7, 2025
3 of 7 checks passed
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.

3 participants