Skip to content

Fix az-arm.js resolved flag and add CI check to enforce generated file sync#840

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/add-pr-check-for-file-sync
Draft

Fix az-arm.js resolved flag and add CI check to enforce generated file sync#840
Copilot wants to merge 2 commits intomainfrom
copilot/add-pr-check-for-file-sync

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

PR #824 manually edited the generated az-arm.js to set resolved: false for XMSSecretInResponse, diverging from the source az-arm.ts which has resolved: true. There was no CI guard to catch this drift.

Changes

  • packages/rulesets/generated/spectral/az-arm.js — Correct resolved: falseresolved: true for XMSSecretInResponse, matching the TypeScript source.

  • .github/workflows/test.yaml — Add a Verify generated files are up-to-date step after rush build and rush regen-ruleindex:

    - name: Verify generated files are up-to-date
      run: git diff --exit-code
      shell: bash

    Fails the build if any committed generated file differs from what the build produces, preventing future drift.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: tejaswiMinnu <14865963+tejaswiMinnu@users.noreply.github.com>
Copilot AI changed the title [WIP] Add PR check to ensure files stay in sync Fix az-arm.js resolved flag and add CI check to enforce generated file sync Mar 3, 2026
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.

2 participants