Skip to content
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.

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

ARM: Add put-operation-response-codes and post-operation-response-codes rules to allow disabling LintDiff rules #369

Merged
merged 6 commits into from
Mar 20, 2024

Conversation

tjprescott
Copy link
Member

@tjprescott tjprescott commented Mar 5, 2024

Closes issues: #360 and #361

REST API Specs
TBD

Azure OpenAPI Validator
TBD

@azure-sdk
Copy link
Collaborator

All changed packages have been documented.

@tjprescott tjprescott force-pushed the arm/putPostResponseCodesRules branch from 74e67a4 to 0144ed1 Compare March 6, 2024 21:46
@azure-sdk
Copy link
Collaborator

@tjprescott tjprescott force-pushed the arm/putPostResponseCodesRules branch 4 times, most recently from 7804a05 to 657536a Compare March 11, 2024 20:52
@tjprescott tjprescott marked this pull request as ready for review March 11, 2024 20:52
@tjprescott tjprescott force-pushed the arm/putPostResponseCodesRules branch from 657536a to 5ad0029 Compare March 11, 2024 21:21
@tjprescott tjprescott force-pushed the arm/putPostResponseCodesRules branch from 5ad0029 to e9f00db Compare March 12, 2024 19:49
@tjprescott tjprescott enabled auto-merge (squash) March 20, 2024 22:14
@tjprescott tjprescott merged commit f04da1c into main Mar 20, 2024
14 checks passed
@tjprescott tjprescott deleted the arm/putPostResponseCodesRules branch March 20, 2024 22:44
tjprescott added a commit that referenced this pull request Mar 21, 2024
Follow up to: #369

Forgot to the put the POST rule in the default ruleset!

---------

Co-authored-by: Timothee Guerin <timothee.guerin@outlook.com>
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