Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update format of generate script console output #67

Merged
merged 1 commit into from
Jan 9, 2021

Conversation

coni2k
Copy link
Contributor

@coni2k coni2k commented Jan 9, 2021

When running generate script with large numbers, there's no way of knowing how many repos are processed and how many left to go. So, I wanted to display an index for each line. Since I started playing with it, I changed the output format a little.

Current output

name,url,language,created_since,updated_since,contributor_count,org_count,commit_frequency,recent_releases_count,updated_issues_count,closed_issues_count,comment_frequency,dependents_count,criticality_score
Hacky,https://github.com/eliaskg/Hacky,Objective-C,97,84,4,2,0.0,0,0,0,0,0,0.10185
flame.js,https://github.com/flamejs/flame.js,JavaScript,107,57,34,5,0.0,0,0,0,0,2,0.22108

New output: no header, each line only shows index, name, url, score

0 - iglo - https://github.com/subosito/iglo - 0.1077
1 - Hacky - https://github.com/eliaskg/Hacky - 0.10185

Alternatively, if we still want to see all the fields, I can keep the old format but only add the index like this:

name,url,language,created_since,updated_since,contributor_count,org_count,commit_frequency,recent_releases_count,updated_issues_count,closed_issues_count,comment_frequency,dependents_count,criticality_score
0 - Hacky,https://github.com/eliaskg/Hacky,Objective-C,97,84,4,2,0.0,0,0,0,0,0,0.10185
1 - flame.js,https://github.com/flamejs/flame.js,JavaScript,107,57,34,5,0.0,0,0,0,0,2,0.22108

Let me know what you think

@coni2k coni2k changed the title Update format for generate script output Update format of generate script output Jan 9, 2021
@inferno-chromium
Copy link
Contributor

Sorry we cannot have this. csv format is needed to quickly import into google sheets, excel, etc and pretty standard way to do stuff. this new format does not help with that.

@coni2k
Copy link
Contributor Author

coni2k commented Jan 9, 2021

The output file itself still stays as csv. This change is only about the output to the console

@coni2k coni2k changed the title Update format of generate script output Update format of generate script console output Jan 9, 2021
@inferno-chromium inferno-chromium merged commit 2757693 into ossf:main Jan 9, 2021
@inferno-chromium
Copy link
Contributor

The output file itself still stays as csv. This change is only about the output to the console

SG

@coni2k coni2k deleted the add-index-field branch January 9, 2021 18:45
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.

None yet

2 participants