Skip to content

Remove Setup-BranchRuleset.ps1 and Fix-BranchRuleset.ps1 (kept canonical in repo-template)#143

Merged
Chris-Wolfgang merged 3 commits into
mainfrom
chore/template-drift-fixes
Apr 30, 2026
Merged

Remove Setup-BranchRuleset.ps1 and Fix-BranchRuleset.ps1 (kept canonical in repo-template)#143
Chris-Wolfgang merged 3 commits into
mainfrom
chore/template-drift-fixes

Conversation

@Chris-Wolfgang

Copy link
Copy Markdown
Owner

Summary

Removes:

  • scripts/Setup-BranchRuleset.ps1
  • scripts/Fix-BranchRuleset.ps1

These are kept canonical only in repo-template since they are run-once at initial repo creation. Once a repo exists, ruleset state is managed via the GitHub API directly — keeping copies of these scripts in every repo just creates drift to maintain.

If you ever need to re-create or repair a ruleset, grab the script from repo-template or use the API directly.

Branch

Named chore/template-drift-fixes so further drift fixes can be added as additional commits to the same PR.

Chris-Wolfgang and others added 3 commits April 28, 2026 21:06
These are run-once scripts for initial repo setup. They are kept
canonical in repo-template only. Once a repo exists, the ruleset
state is managed via the GitHub API; keeping copies of the scripts
in every repo just creates drift.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Pure staleness drift -- mostly missing recent canonical additions
(Validate release tag step, 3.1.x in setup-dotnet, Node 24 actions
in Hawsey/Log-Compressor) and minor whitespace differences.

No unique features in any repo's existing release.yaml that needed
preserving (IEquatable-Extensions and console-app-template are
structurally divergent and intentionally excluded from this sync).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Restores Directory.Build.props to canonical (centralizes 6
  analyzers, drops vestigial UsingMicrosoftNETSdk condition on
  the AdditionalFiles ItemGroup since all csprojs are SDK-style).
- Removes duplicate analyzer ItemGroup from all 18 csprojs (src,
  test, 8 net4.8 examples, 8 net8.0 examples) — now inherited.
- Refreshes .editorconfig (#325 + #326).
- Refreshes pr.yaml (#322 + #323 + #324).
- Refreshes docfx.yaml (#327).
- Adds coverlet.runsettings (new from #324).

Release build clean: 0 warnings, 0 errors across all TFMs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.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.

1 participant