codecov / codecov/codecov-rs

associate measurements with a specific coverage file, not just a raw upload

Open
#37 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
4
Forks
3
PR merge metrics
No merged PRs in 30d

Description

currently measurements record which raw upload they came from, but a raw upload can contain several separate coverage data files. it would be nice to, for example, tie one measurement to the lcov file and another measurement to the cobertura file

we could accomplish this by replacing raw_upload_id in all of the measurement tables with a coverage_file_id which refers to some new coverage_file table:

create table coverage_file (
    id INTEGER PRIMARY KEY, -- random
    path VARCHAR NOT NULL, -- e.g. "./lcov.info"
    format VARCHAR NOT NULL, -- e.g. "lcov", "cobertura"
    raw_upload_id INTEGER REFERENCES raw_upload(id) NOT NULL
);

existing pyreports will not have this information, so we can just treat them as coming from a single coverage file of format "pyreport"

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the measurement tables, raw_upload schema, and pyreport handling in the repository. Trace how uploads and measurements are persisted, then determine the migration and compatibility work needed so measurements reference coverage files while existing pyreports remain supported.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
databases
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.