-
Notifications
You must be signed in to change notification settings - Fork 78
feat: add link to GitHub statistics from deployments page #1215
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
base: main
Are you sure you want to change the base?
Conversation
This uses https://github.com/jgehrcke/github-repo-stats so that we can track usage beyond the 14 day window provided by GitHub. Signed-off-by: Derek Hower <[email protected]>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1215 +/- ##
=======================================
Coverage 46.05% 46.05%
=======================================
Files 11 11
Lines 4942 4942
Branches 1345 1345
=======================================
Hits 2276 2276
Misses 2666 2666
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| <h3>Repository development</h3> | ||
| <ul> | ||
| <li><a href="<%= ENV["PAGES_URL"] %>/reuse_bom.txt">Reuse Bill of Materials (LICENSE information)</a></li> | ||
| <li><a href="https://dhower-qc.github.io/udb-stats/riscv-software-src/riscv-unified-db/latest-report/report.html">GitHub usage statistics (updated daily)</a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason not to have the workflow as part of this repo and use the UDB domain for this instead of your personal GitHub account? Looks like having it all in one repository is supported: https://github.com/jgehrcke/github-repo-stats/wiki/Tutorial.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The stats get published as a GitHub Pages site. Since we already have a Pages site for UDB, we need another repo to host the stats.
It would probably be better to move the stats site under riscv-software-src. @rpsene, can you help with that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dhower-qc yes, we can move, but what about the https://github.com/riscv-software-src/riscv-unified-db/graphs/traffic ? Looks like it provides the same data you have at https://dhower-qc.github.io/udb-stats/riscv-software-src/riscv-unified-db/latest-report/report.html.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The GitHub-provided data only provides a 14 day lookback. The data pointed to by this PR provides unlimited lookback.
This uses https://github.com/jgehrcke/github-repo-stats so that we can track usage beyond the 14 day window provided by GitHub.