Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

[spike] Aggregate commit authorship signal#50727

Closed
cbart wants to merge 11 commits into
mainfrom
proto-cb/aggregate-commit-spike
Closed

[spike] Aggregate commit authorship signal#50727
cbart wants to merge 11 commits into
mainfrom
proto-cb/aggregate-commit-spike

Conversation

@cbart

@cbart cbart commented Apr 17, 2023

Copy link
Copy Markdown
Contributor

This is a prototype of aggregate commit authorship signals.

This is how it works:

  1. Database is central and contains both the individual signals as well as aggregates.
  2. Aggregates are updated with stored procedure as new data appears.
  3. The API serves ownership signal based on commit authorship for every directory with a single database query.
  4. Code intel tooling schedules scanning repo and adding relevant commits to the history.

TODO

  • Database
  • GraphQL API wired to the database
  • UI
  • Pulling most recent commit data
    • and adding them to the database
  • Code intel like indexing

Caveats

  • The commit adding database code does not consider root directory of repo yet. - simple fix.

Test plan

Spike spike

@cla-bot cla-bot Bot added the cla-signed label Apr 17, 2023
cbart added a commit that referenced this pull request Apr 20, 2023
@cbart cbart closed this May 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant