Skip to content

test: 100 packages Weighted 12m (DO NOT MERGE) - #57904

Draft
m-nash wants to merge 19 commits into
feature/weighted-test-batchingfrom
test/batch-w12-100pkg
Draft

test: 100 packages Weighted 12m (DO NOT MERGE)#57904
m-nash wants to merge 19 commits into
feature/weighted-test-batchingfrom
test/batch-w12-100pkg

Conversation

@m-nash

@m-nash m-nash commented Apr 7, 2026

Copy link
Copy Markdown
Member

Batching validation test PR.

@github-actions

github-actions Bot commented Apr 10, 2026

Copy link
Copy Markdown

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Azure.ResourceManager.DesktopVirtualization
Azure.ResourceManager.Chaos

@m-nash
m-nash requested a review from Copilot May 14, 2026 17:22

Copilot AI left a comment

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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR appends a "Test comment for batching validation - DO NOT MERGE" line to the end of 100 source files across the Azure SDK for .NET. The PR title and description explicitly state it should not be merged; it is intended only for batching validation testing.

Changes:

  • Appends an identical trailing comment line to 100 .cs files across many SDK packages.
  • One file (ArmMonitorModelFactory.cs) additionally has its closing } re-indented/renumbered while the trailing comment is added outside the namespace/class scope.
  • No functional code changes; only comments are added.

Reviewed changes

Copilot reviewed 87 out of 99 changed files in this pull request and generated no comments.

File Description
100 .cs files across sdk/** Each file gets an appended // Test comment for batching validation - DO NOT MERGE line outside any type/namespace.
sdk/monitor/Azure.ResourceManager.Monitor/src/Customized/ArmMonitorModelFactory.cs Same trailing comment plus a closing-brace line modification shown in the diff.
Comments suppressed due to low confidence (1)

sdk/monitor/Azure.ResourceManager.Monitor/src/Customized/ArmMonitorModelFactory.cs:1

  • Unlike the other 99 files where the test comment is appended after the final closing brace with a separating blank line, here the diff shows the closing } line being removed and re-added as line 46 with the comment immediately following on line 47 (no blank line). Verify the file still ends with a newline and that no namespace/class brace was inadvertently dropped during the edit; the asymmetry vs. the other files suggests an editing mistake.

m-nash and others added 16 commits May 21, 2026 15:33
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Hi m-nash. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

@github-actions github-actions Bot added the no-recent-activity There has been no recent activity on this issue. label Jul 31, 2026
@weikanglim
Wei Lim (weikanglim) force-pushed the feature/weighted-test-batching branch from 461261f to b941898 Compare July 31, 2026 21:34
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: e5e0d1d6-f002-4bd1-9156-55ec649d89cf
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: e5e0d1d6-f002-4bd1-9156-55ec649d89cf
@github-actions github-actions Bot removed the no-recent-activity There has been no recent activity on this issue. label Jul 31, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 19db450c-9af7-474e-b3f1-640d1f968126
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