Skip to content

Improving test tooling before tackling overrides#70

Merged
perryqh merged 10 commits intomainfrom
ph.overrides-prep
Aug 18, 2025
Merged

Improving test tooling before tackling overrides#70
perryqh merged 10 commits intomainfrom
ph.overrides-prep

Conversation

@perryqh
Copy link
Copy Markdown
Contributor

@perryqh perryqh commented Aug 17, 2025

  • from-codeowners flag for for-file command will return the team as derived from .github/CODEOWNERS
  • hidden crosscheck-owners walks the entire repo and compares for-file results against for-file --from-codeowners
  • new fixture containing overrides

Copy link
Copy Markdown
Contributor

@jackboberg jackboberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Crosscheck is a nice addition!

@perryqh perryqh merged commit fc4e148 into main Aug 18, 2025
8 checks passed
@github-project-automation github-project-automation Bot moved this from Triage to Done in Modularity Aug 18, 2025
@perryqh perryqh deleted the ph.overrides-prep branch September 5, 2025 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants