chaoss / chaoss/CollectOSS

Epic: Fix large code duplication

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

Description

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

---

Several observations have bene made about ways in which major components/functions of the codebase are duplicated, requireing changes to be made in more than one place.

- #121
- augurlabs/augur#3357 (helps fix duplication in reading from config
~~- The get_section in lib.py (more reading from config dupe not addressed by the above) augurlabs/augur#3357 ~~
- augur backend start cli function and augur api start cli function seem to be very similar...
- bulk_insert_dicts and session.insert_data code seems very similar (found since https://github.com/chaoss/augur/pull/3342/changes updated both of them)
- the get_value in lib.py is duplicated with config stuff
- facade has an entire config helper to itself... seems redundant

Is there a historical reason for this?

@sgoggins

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.