synthesized-io / insight

🧿 Metrics & Monitoring of Datasets
BSD 3-Clause "New" or "Revised" License
12 stars 0 forks source link

Feature: added Result.run_id #141

Closed marqueewinq closed 10 months ago

marqueewinq commented 10 months ago

Currently the insight package populates the Results without any grouping except created_at date. I'm adding Result.run_id to fix that.

codecov[bot] commented 10 months ago

Codecov Report

Merging #141 (8fb2211) into master (4ee4908) will increase coverage by 0.0%. The diff coverage is 100.0%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #141   +/-   ##
======================================
  Coverage    79.5%   79.5%           
======================================
  Files          11      11           
  Lines         888     890    +2     
  Branches      135     135           
======================================
+ Hits          706     708    +2     
  Misses        140     140           
  Partials       42      42           
Files Changed Coverage Δ
src/insight/database/schema.py 100.0% <100.0%> (ø)
src/insight/metrics/base.py 74.8% <100.0%> (+0.1%) :arrow_up:
marqueewinq commented 10 months ago

28.6% 28.6% Coverage 0.0% 0.0% Duplication

Perhaps we need to disable coverage for migrations

sonarcloud[bot] commented 10 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication