chaoss / chaoss/CollectOSS

Remove potentially legacy working commits call/table

Open
#117 4 comments 0 reactions 0 assignees View on GitHub
database tech debt
Dominant language
Python
Stars
13
Forks
17
Avg merge
6h 59m
Merged PRs (30d)
1

Description

> [!NOTE]
> Migrated from [augurlabs/augur#3323](https://github.com/augurlabs/augur/issues/3323)
> Originally opened by `@MoralCode` on 2025-10-21

---

https://github.com/chaoss/augur/blob/c5eeb072ae5c541e5fbb56295b3afab5dffaf4b0/augur/tasks/git/facade_tasks.py#L335C5-L335C49

this line prompted a disussion with @IsaacMilarky

It seems like this line is from legacy facade where it would collect into a working_commits table (so it could pick up a partial collection where it left off) and then at the end it would transfer them to the commits table.

As far as i can see, analyze_commits_in_parallel calls this https://github.com/chaoss/augur/blob/c5eeb072ae5c541e5fbb56295b3afab5dffaf4b0/augur/application/db/lib.py#L212

which uses the ORM Commit object https://github.com/chaoss/augur/blob/main/augur/application/db/models/augur_data.py#L1261
to write directly to the `commits` table

so this may need to be removed along with the table it refers to (although maybe an audit is necessary)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.