Merge pull request #714 from microsoftgraph/ci/redocly-validation-fail

ci: removes continue on error for OAS validation
microsoftgraph · Nov 15, 2024 · 014d084 · 014d084
2 parents 051f0e0 + 483cd29
commit 014d084
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/.github/workflows/metadata-parser-validation.yml b/.github/workflows/metadata-parser-validation.yml
@@ -41,7 +41,6 @@ jobs:
 
     - name: Validate description with redocly
       uses: fluximus-prime/redocly-cli-github-action@v1
-      continue-on-error: true # temporary workaround since our descriptions are invalid at the moment
       with:
         args: 'lint transformed_${{ matrix.version }}_${{ matrix.settings }}_metadata.yml --skip-rule operation-4xx-response --skip-rule no-server-trailing-slash --skip-rule no-unused-components --skip-rule security-defined --skip-rule info-license-url --skip-rule info-license --skip-rule no-empty-servers --skip-rule operation-summary --skip-rule tag-description --max-problems 1000'
 

diff --git a/.github/workflows/openapi-parser-validation.yml b/.github/workflows/openapi-parser-validation.yml
@@ -40,6 +40,5 @@ jobs:
 
     - name: Validate description with redocly
       uses: fluximus-prime/redocly-cli-github-action@v1
-      continue-on-error: true # temporary workaround since our descriptions are invalid at the moment
       with:
         args: 'lint openapi/${{ matrix.version }}/${{ matrix.settings }}.yaml --skip-rule operation-4xx-response --skip-rule no-server-trailing-slash --skip-rule no-unused-components --skip-rule security-defined --skip-rule info-license-url --skip-rule info-license --skip-rule no-empty-servers --skip-rule operation-summary --skip-rule tag-description --max-problems 1000'