Skip to content

Add module initializer for Verify.SourceGenerators #60547

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Feb 24, 2025
Merged

Add module initializer for Verify.SourceGenerators #60547

merged 1 commit into from
Feb 24, 2025

Conversation

captainsafia
Copy link
Member

Realized I missed the module initializer for these snapshot tests while working on the implementation. The current verified.txt file gets removed since the source generator isn't currently emiting anything.

@Copilot Copilot AI review requested due to automatic review settings February 21, 2025 23:13
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions label Feb 21, 2025
@captainsafia captainsafia enabled auto-merge (squash) February 21, 2025 23:13
Copilot
Copilot AI reviewed Feb 21, 2025
View reviewed changes
Copy link
Contributor

@Copilot 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.

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

@captainsafia captainsafia added area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework and removed area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions labels Feb 23, 2025
@captainsafia captainsafia merged commit 9175048 into main Feb 24, 2025
27 checks passed
@captainsafia captainsafia deleted the fix-vsg-test branch February 24, 2025 21:50
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview3 milestone Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@BrennanConroy BrennanConroy BrennanConroy approved these changes

@halter73 halter73 Awaiting requested review from halter73 halter73 is a code owner

Assignees
No one assigned
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Milestone
10.0-preview3
Development

Successfully merging this pull request may close these issues.
2 participants
@captainsafia @BrennanConroy