Skip to content

Align fork metrics with CHAOSS Technical Fork definition#3642

Open
HimasreeKolathur24 wants to merge 1 commit intochaoss:mainfrom
HimasreeKolathur24:fix/2603-technical-fork
Open

Align fork metrics with CHAOSS Technical Fork definition#3642
HimasreeKolathur24 wants to merge 1 commit intochaoss:mainfrom
HimasreeKolathur24:fix/2603-technical-fork

Conversation

@HimasreeKolathur24
Copy link

@HimasreeKolathur24 HimasreeKolathur24 commented Feb 2, 2026

Description

  • This PR aligns Augur’s existing fork metrics (forks and fork_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

  • No functional changes were made; this PR only updates metric documentation to align with the CHAOSS canonical definition.
  • No database or API behavior changes are introduced.

Signed commits

  • Yes, I signed my commits.

@HimasreeKolathur24 HimasreeKolathur24 force-pushed the fix/2603-technical-fork branch 2 times, most recently from 98f8705 to 95290f4 Compare February 2, 2026 16:09
@sgoggins sgoggins added the documentation Updates documentation label Feb 2, 2026
@MoralCode
Copy link
Contributor

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?

Copy link
Collaborator

@shlokgilda shlokgilda left a comment

Choose a reason for hiding this comment

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

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:

  1. GitHub GraphQL API forkCountcollected here
  2. Stored in repo_info.fork_countline 203
  3. 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>
@HimasreeKolathur24
Copy link
Author

@MoralCode, @shlokgilda,
Thanks for the review and suggestions! I’ve fixed the docstring indentation for both forks and fork_count, squashed the commits into a single clean commit, and addressed the formatting feedback. No functional changes were made.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Updates documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Technical Fork metric API

4 participants