Skip to content
New issue

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

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

Already on GitHub? # to your account

Updated some content on the schema/augur_full.sql file. #2821

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Preshh0
Copy link
Contributor

@Preshh0 Preshh0 commented Jun 13, 2024

Description

  • Replaced "bus factor" with "contributor absence factor" as the metric has been renamed.

This PR fixes #

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@geekygirldawn geekygirldawn requested a review from sgoggins June 17, 2024 07:12
@geekygirldawn
Copy link
Member

geekygirldawn commented Jun 17, 2024

I'm not sure how we should be handling the changes to the name of the metric within Augur, but @sgoggins or someone on the Augur maintainer team should be able to provide some guidance.

@Preshh0
Copy link
Contributor Author

Preshh0 commented Jun 17, 2024

Alright, @geekygirldawn.

@@ -3744,7 +3744,7 @@ COPY augur_data.chaoss_metric_status (cms_id, cm_group, cm_source, cm_type, cm_b
43 experimental githubapi metric implemented unimplemented f /api/unstable/<owner>/<repo>/githubapi/pull_requests_merged \N Pull Requests Merged experimental "pull_requests_merged" Insight Worker 0.0.1 githubapi 2019-06-20 22:50:24 \N
44 experimental githubapi metric implemented unimplemented f /api/unstable/<owner>/<repo>/githubapi/pull_requests_open \N Pull Requests Open experimental "pull_requests_open" Insight Worker 0.0.1 githubapi 2019-06-20 22:50:25 \N
45 experimental githubapi metric implemented unimplemented t /api/unstable/<owner>/<repo>/githubapi/repository_size \N Repository Size experimental "repository-size" Insight Worker 0.0.1 githubapi 2019-06-20 22:50:27 \N
46 experimental githubapi metric implemented implemented t /api/unstable/<owner>/<repo>/bus_factor \N Bus Factor experimental "bus-factor" Insight Worker 0.0.1 githubapi 2019-06-20 22:50:28 \N
46 experimental githubapi metric implemented implemented t /api/unstable/<owner>/<repo>/contributor_absence_factor \N Contributor Absence Factor experimental "contributor-absence-factor" Insight Worker 0.0.1 githubapi 2019-06-20 22:50:28 \N
Copy link
Member

Choose a reason for hiding this comment

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

@Preshh0 : Unfortunately making this change will be a bit more work. I know for sure there are files in :
augur/api and docs/ that will need to be addressed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alrighty then. Do I close it entirely? Or leave it open and wait until whenever there is time to work on it? @sgoggins

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants