Skip to content

Remove committed settings.local.json, add contributor template, tighten .gitignore#512

Draft
Copilot wants to merge 10 commits into
mainfrom
copilot/sub-pr-509
Draft

Remove committed settings.local.json, add contributor template, tighten .gitignore#512
Copilot wants to merge 10 commits into
mainfrom
copilot/sub-pr-509

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 23, 2026

settings.local.json (Claude IDE permissions config) was accidentally committed — machine-specific and a potential source of permission drift across contributors.

Changes

  • Untrack settings.local.json — removed from git index; file remains locally but is no longer committed
  • Add settings.json.example — checked-in template at WDAC-Policy-Wizard/app/.claude/settings.json.example documenting the expected Claude permissions:
    {
      "permissions": {
        "allow": [
          "Bash(dotnet add:*)",
          "Bash(dotnet build:*)",
          "Bash(dotnet test:*)",
          "Bash(dotnet sln:*)"
        ]
      }
    }
  • Update .gitignore — replaced the broad .claude/ directory ignore with *.local.json, allowing the example template to be tracked while keeping all *.local.json files ignored

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

jgeurten and others added 9 commits January 29, 2026 18:42
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Initial plan

* Fix stale comments in UpdateVersion test cases to match actual implementation

Co-authored-by: jgeurten <23045608+jgeurten@users.noreply.github.com>
Agent-Logs-Url: https://github.com/MicrosoftDocs/WDAC-Toolkit/sessions/6c0a8b8c-31b6-4779-b9a7-baa99617b602

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jgeurten <23045608+jgeurten@users.noreply.github.com>
Copilot AI changed the title [WIP] [WIP] Address feedback on test cases for the Wizard Remove committed settings.local.json, add contributor template, tighten .gitignore Mar 23, 2026
Copilot AI requested a review from jgeurten March 23, 2026 15:36
Base automatically changed from add-test-cases to main March 23, 2026 15:44
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