Skip to content

Add operation-aware C# body suffix linting - #5

Closed
haiyuazhang wants to merge 2 commits into
haiyzhan/csharp-model-suffix-rulesfrom
haiyzhan/csharp-patch-body-suffix-rule
Closed

Add operation-aware C# body suffix linting#5
haiyuazhang wants to merge 2 commits into
haiyzhan/csharp-model-suffix-rulesfrom
haiyzhan/csharp-patch-body-suffix-rule

Conversation

@haiyuazhang

Copy link
Copy Markdown
Owner

Summary

  • make csharp-model-suffix operation-aware for direct PATCH and PUT/POST bodies
  • use Content for nested request models and suppress diagnostics for conflicting request/response roles
  • add code fixes, documentation, and comprehensive coverage for body-shape and reuse scenarios

Addresses the design for Azure#4447. This PR targets the batch2 branch for isolated review before merging into Azure#4867.

Validation

  • targeted rule suite: 53 tests passed
  • rule coverage: 95.94% branches, 100% lines
  • TCGC dependency build passed
  • repository format and lint passed

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 12e46e52-385e-444f-a3fc-ee8cd7d97c83
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 12e46e52-385e-444f-a3fc-ee8cd7d97c83
@haiyuazhang

Copy link
Copy Markdown
Owner Author

Superseded by Azure#5058.

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.

1 participant