Add files report #139
Merged
Add files report #139
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
laserkelvin
approved these changes
Dec 20, 2024
codebasin/report.py
Outdated
| return f"{x/10**6:.1f}M" | ||
| elif digits <= 12: | ||
| return f"{x/10**9:.1f}G" | ||
| return "??????" |
There was a problem hiding this comment.
Maybe ******* instead?
There was a problem hiding this comment.
Fixed in 987692c.
| if not isinstance(x, numbers.Integral): | ||
| raise TypeError("x must be Integral") | ||
|
|
||
| digits = len(str(x)) |
There was a problem hiding this comment.
Can you use log10 instead LOL
Introduces a new files report that visualizes the entire code base as a tree with a format inspired by the "tree" utility. Information about the platforms and SLOC used by each node in the tree is printed similar to tree's "metafirst" output, in the format: [Platforms | Used SLOC / Total SLOC | Code Divergence | Code Utilization] Colors and formatting are all hard-coded in this initial version, but the long-term plan is to allow configuration via the .cbi/config file. Signed-off-by: John Pennycook <[email protected]>
Signed-off-by: John Pennycook <[email protected]>
Signed-off-by: John Pennycook <[email protected]>
Signed-off-by: John Pennycook <[email protected]>
Signed-off-by: John Pennycook <[email protected]>
Listing out the column names in full allows us to convey what the meta-information shows more clearly than adding short column headers, and does not require any complicated justification/padding logic for the different columns. Signed-off-by: John Pennycook <[email protected]>
Signed-off-by: John Pennycook <[email protected]>
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Introduces a new files report that visualizes the entire code base as a tree with a format inspired by the "tree" utility.
Information about the platforms and SLOC used by each node in the tree is printed similar to tree's "metafirst" output, in the format:
Colors and formatting are all hard-coded in this initial version, but the long-term plan is to allow configuration via the .cbi/config file.
Related issues
N/A
Proposed changes
FileTreeobject to represent a directory of files + associated platform information hierarchically.filesreport that builds aFileTreefor aCodeBaseand prints a colored version to the terminal.FileTreeand associated functionality.@laserkelvin - I can only apologize for this being such a big diff... The feature proved really complicated to get right, and even after a bunch of refactoring I couldn't really find a good way to split things up for review. I completely understand if this takes you a while!
Here's an example of the current output: