Skip to content

chore: add @wbreza to CODEOWNERS#111

Merged
github-actions[bot] merged 1 commit into
mainfrom
add-wbreza-codeowners
Mar 11, 2026
Merged

chore: add @wbreza to CODEOWNERS#111
github-actions[bot] merged 1 commit into
mainfrom
add-wbreza-codeowners

Conversation

@spboyer

@spboyer spboyer commented Mar 11, 2026

Copy link
Copy Markdown
Member

Adds @wbreza as a code reviewer for all files in the repository.

Copilot AI review requested due to automatic review settings March 11, 2026 17:59
@github-actions github-actions Bot enabled auto-merge (squash) March 11, 2026 18:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds @wbreza as a default code owner so they are auto-requested as a reviewer on PRs across the repository.

Changes:

  • Update .github/CODEOWNERS default (*) owners to include @wbreza.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread .github/CODEOWNERS

@wbreza wbreza left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Code Review Summary

Recommendation: APPROVE

What Changed

  • Added @wbreza (Wallace Breza) as a code owner for all files in the repository
  • Single line change to .github/CODEOWNERS file

Verification Completed

Valid user: @wbreza is a valid GitHub user account
Repository access: User has collaborator access to microsoft/waza
CODEOWNERS syntax: Correct format (wildcard pattern with space-separated usernames)
No unrelated changes: Only the intended modification to CODEOWNERS
Comment updated: In-file documentation reflects 4 reviewers (though comment could be updated to mention @wbreza specifically)

Findings

No significant issues found in the reviewed changes.

Minor observation (not blocking): The comment on line 2 mentions "All three will be auto-assigned" but there are now four code owners. This is cosmetic and doesn't affect functionality, but could optionally be updated to "All will be auto-assigned" or "All four will be auto-assigned" for accuracy.

Conclusion

This is a clean, well-formed change that correctly adds a new code owner. The change is safe to merge.

wbreza
wbreza previously approved these changes Mar 11, 2026

@wbreza wbreza left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM — clean CODEOWNERS addition. ✅

Copilot AI review requested due to automatic review settings March 11, 2026 18:37
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


You can also share your feedback on Copilot code review. Take the survey.

@github-actions github-actions Bot merged commit 9b9084c into main Mar 11, 2026
5 checks passed
richardpark-msft pushed a commit to richardpark-msft/waza that referenced this pull request Mar 13, 2026
Adds a new `tokens compare` subcommand that compares token counts in a
directory between git refs, enabling users to track how token usage
changes across commits or between committed and uncommitted work. Closes
microsoft#51

## Usage

```bash
# Compare HEAD to working tree
waza tokens compare

# Compare a specific ref to working tree
waza tokens compare HEAD~3

# Compare two refs
waza tokens compare main feature-branch

# JSON output
waza tokens compare --format json

# Include unchanged files
waza tokens compare --show-unchanged
```

## Changes

### New files

| File | Description |
| -------------------------------------------- |
-----------------------------------------------------------------------------------------------------------
|
| `cmd/waza/tokens/compare.go` | Command implementation: ref resolution,
file diffing, token counting, table and JSON output |
| `cmd/waza/tokens/compare_test.go` | Tests covering
added/modified/removed files, multiple ref combinations,
`--show-unchanged`, and JSON output |
| `cmd/waza/tokens/internal/git/git.go` | Git helpers: `IsInRepo`,
`GetFilesFromRef`, `GetFileFromRef`, `RefExists` |

## Design

- **Ref handling:** With no args, compares `HEAD → WORKING` i.e., the
working tree. One arg compares that ref to working tree. Two args
compares first to second.
- **File detection:** Uses `git ls-files` for the working tree and `git
ls-tree` for committed refs, filtering to `.md`/`.mdx` files.
- **Edge cases:** Handles empty repos (no HEAD), deleted working tree
files, and files that exist in only one ref.
- **Output formats:** Table (default) with emoji status indicators, or
structured JSON with per-file and summary statistics.
- **Per-file stats:** Token count, character count, line count, absolute
diff, percent change, and status
(`added`/`removed`/`modified`/`unchanged`).

## Example Output
```
📊 Token Comparison: HEAD → WORKING
File                  Before     After      Diff  Status
------------------------------------------------------------------
README.md                 11        14        +3  📈
references/spec.md         7         7         0  ➡️
unchanged.md               1         1         0  ➡️
------------------------------------------------------------------
Total                     19        22        +3  15.8%
📋 Summary:
    Added: 0, Removed: 0, Modified: 1
    Increased: 1, Decreased: 0
```
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.

5 participants