Skip to content

Stop uploading accepted line fingerprints#22180

Merged
alexsong-oai merged 1 commit into
mainfrom
alexs/accepted-line-counts-only
May 11, 2026
Merged

Stop uploading accepted line fingerprints#22180
alexsong-oai merged 1 commit into
mainfrom
alexs/accepted-line-counts-only

Conversation

@alexsong-oai
Copy link
Copy Markdown
Collaborator

Summary

  • keep accepted-line diff parsing and fingerprint hashing logic locally
  • stop uploading path/line hash fingerprints in the accepted-line analytics event payload
  • keep aggregate accepted added/deleted line counts in the event

Testing

  • just fmt
  • cargo test -p codex-analytics
  • just fix -p codex-analytics

@alexsong-oai alexsong-oai force-pushed the alexs/accepted-line-counts-only branch from db66f4f to c40e247 Compare May 11, 2026 17:59
@alexsong-oai alexsong-oai marked this pull request as ready for review May 11, 2026 19:39
@alexsong-oai alexsong-oai merged commit bb6134c into main May 11, 2026
27 checks passed
@alexsong-oai alexsong-oai deleted the alexs/accepted-line-counts-only branch May 11, 2026 22:41
@github-actions github-actions Bot locked and limited conversation to collaborators May 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants