Skip to content

Conversation

@zbedforrest
Copy link
Collaborator

Related PR(s)

Related Issue(s)

#10

Summary / Background

Adds a new cli command to print out an ownership map in JSON format. It can be used to print a map from owners to their files, or from files to their owners.

A future improvement (or maybe in this branch, depending on discussions) would be to print files in their directory tree structure. Right now, the command prints out flat maps of file names, so there is a lot of repetition in the strings of the parent directories.

@gemini-code-assist
Copy link

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@pooyaw
Copy link

pooyaw commented Jul 16, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@github-actions
Copy link

github-actions bot commented Jul 16, 2025

Codeowners approval required for this PR:

@github-actions github-actions bot requested a review from BakerNet July 16, 2025 23:48
Copy link
Collaborator

@BakerNet BakerNet left a comment

Choose a reason for hiding this comment

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

Need to run ./scripts/covbadge and commit but LGTM

@zbedforrest
Copy link
Collaborator Author

zbedforrest commented Jul 17, 2025

Need to run ./scripts/covbadge and commit but LGTM

sweet, thanks for looking! I want to tidy a few things up, then will ping you / take it out of draft mode

also, I kept it really simple so far... the outputs are flat, for one thing. Maybe a future improvement would be to output the files in a nested directory structure so we dont print out the parent dirs 10000 times. But i didn't want to get into that unless there is a use case that calls for it

@BakerNet BakerNet merged commit 4dd3f5f into main Jul 17, 2025
11 of 12 checks passed
@BakerNet BakerNet deleted the feature/ownership-map-command branch July 17, 2025 17:06
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.

4 participants