From 5f8a448c46c2d1fad8e46ea5b0dcd7d50c38aaf0 Mon Sep 17 00:00:00 2001 From: Tejaswi Salaigari Date: Wed, 4 Sep 2024 16:08:14 -0500 Subject: [PATCH] disable rule PatchPropertiesCorrespondToPutProperties (#737) * disable rule * trailing space --- CONTRIBUTING.md | 5 +---- packages/rulesets/generated/spectral/az-arm.js | 1 + packages/rulesets/package.json | 2 +- packages/rulesets/src/spectral/az-arm.ts | 1 + 4 files changed, 4 insertions(+), 5 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index c9887cfb9..96365b347 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -390,10 +390,7 @@ Once your PR is merged: - Schedule a [Prod build] from the `main` branch. - **!!! IMPORTANT !!!**: If you are updating [packages/azure-openapi-validator/autorest], see [this section](#synchronize-with-changes-to-openapi-alps-repository). -- Once the build is complete, schedule a [Prod npm release] from that build. - You may need to get an approval for the release from the appropriate Azure SDK EngSys team members. -- Note that sometimes the npm release may report failure even when it succeeded. This is because sometimes it tries to - publish package twice and succeeds only on the first try. +- You may need to get an approval for the release from the appropriate [Azure SDK EngSys team members](https://teams.microsoft.com/l/channel/19%3A59dbfadafb5e41c4890e2cd3d74cc7ba%40thread.skype/Engineering%20System%20%F0%9F%9B%A0%EF%B8%8F?groupId=3e17dcb0-4257-4a30-b843-77f47f1d4121). - Verify the release worked by new versions of the appropriate packages being released to npm. See [README `packages` section]. You can also look at the release build log. diff --git a/packages/rulesets/generated/spectral/az-arm.js b/packages/rulesets/generated/spectral/az-arm.js index d8d773109..69938e073 100644 --- a/packages/rulesets/generated/spectral/az-arm.js +++ b/packages/rulesets/generated/spectral/az-arm.js @@ -3458,6 +3458,7 @@ const ruleset = { description: "PATCH request body must only contain properties present in the corresponding PUT request body, and must contain at least one property.", message: "{{error}}", severity: "error", + stagingOnly: true, resolved: true, formats: [oas2], given: ["$[paths,'x-ms-paths'].*"], diff --git a/packages/rulesets/package.json b/packages/rulesets/package.json index da593e3c0..21ae11b5b 100644 --- a/packages/rulesets/package.json +++ b/packages/rulesets/package.json @@ -1,6 +1,6 @@ { "name": "@microsoft.azure/openapi-validator-rulesets", - "version": "2.1.0", + "version": "2.1.1", "description": "Azure OpenAPI Validator", "main": "dist/index.js", "scripts": { diff --git a/packages/rulesets/src/spectral/az-arm.ts b/packages/rulesets/src/spectral/az-arm.ts index d58a511c8..0aab7fa39 100644 --- a/packages/rulesets/src/spectral/az-arm.ts +++ b/packages/rulesets/src/spectral/az-arm.ts @@ -459,6 +459,7 @@ const ruleset: any = { "PATCH request body must only contain properties present in the corresponding PUT request body, and must contain at least one property.", message: "{{error}}", severity: "error", + stagingOnly: true, resolved: true, formats: [oas2], given: ["$[paths,'x-ms-paths'].*"],