Skip to content

Comments

Owners-only option for codeowners owner#26

Merged
timoschinkel merged 2 commits intomainfrom
feature/owner-only
May 28, 2025
Merged

Owners-only option for codeowners owner#26
timoschinkel merged 2 commits intomainfrom
feature/owner-only

Conversation

@timoschinkel
Copy link
Owner

@timoschinkel timoschinkel commented May 28, 2025

Add the option -o/--owner-only to codeowners owner that will only display the found owners and suppresses the other output like filename.

solves #25

Addendum: Change vimeo/psalm requirement from ^6.0 to ~6.4.0; version 6.5.0 enables MissingOverrideAttribute for all versions, but is also introducing some interesting choices about PHP versioning - see vimeo/psalm#11270 - which made that I cannot test this locally. This is the smallest change to make the pull request pass inspections. The new rules/latest version of Psalm will be added/applied in a separate pull request.

Add the option `-o`/`--owner-only` to `codeowners owner` that will only display the found owners and suppresses the other output like filename.

solves #25
Change `vimeo/psalm` requirement from `^6.0` to `~6.4.0`; version `6.5.0` adds a new rule, but is also introduces some interesting choices about PHP versioning - see vimeo/psalm#11270. This is the smallest change to make the pull request pass inspections. The new rules will be added/applied in a separate pull request.
@timoschinkel timoschinkel marked this pull request as ready for review May 28, 2025 11:32
@timoschinkel timoschinkel merged commit 0f324f8 into main May 28, 2025
8 checks passed
@timoschinkel timoschinkel deleted the feature/owner-only branch May 28, 2025 11:33
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