Align fork metrics with CHAOSS Technical Fork definition#3642
Align fork metrics with CHAOSS Technical Fork definition#3642HimasreeKolathur24 wants to merge 1 commit intochaoss:mainfrom
Conversation
98f8705 to
95290f4
Compare
|
Im a little unsure how these new docs relate to the actual functionality, if we are changing the docs, I want to make sure they align with the actual implementation. If those endpoints dont actually represent the technical fork metric, that would need fixing before this merges. Do you have any evidence that these endpoints truly do implement this metric? Can any other maintainers vouch for this? |
shlokgilda
left a comment
There was a problem hiding this comment.
Yeah, these endpoints do implement the CHAOSS Technical Fork metric correctly.
I think this is the data flow. Didn't really dove deep into it though:
- GitHub GraphQL API
forkCount→ collected here - Stored in
repo_info.fork_count→ line 203 - These metrics query that column
GitHub's forkCount only tracks server-side forks. This matches the CHAOSS definition - platform-native forks excluding local clones.
@HimasreeKolathur24 Left some line comments on formatting issues (docstring indentation).
Signed-off-by: HimasreeKolathur24 <himasrikolathur@gmail.com>
074d3d7 to
042c814
Compare
|
@MoralCode, @shlokgilda, |
Description
forksandfork_count) with the CHAOSS Technical Fork canonical definition by updating their documentation. The metrics now explicitly describe platform-native technical forks as defined by CHAOSS, without changing any existing logic or database schema.This PR fixes #2603
Notes for Reviewers
Signed commits