Skip to content

Conversation

@yngve-sk
Copy link
Contributor

@yngve-sk yngve-sk commented Dec 19, 2025

Should go in after #12528 . Prerequisite for #11513

@yngve-sk yngve-sk added this to SCOUT Dec 19, 2025
@yngve-sk yngve-sk moved this to Ready for Review in SCOUT Dec 19, 2025
@codecov-commenter
Copy link

codecov-commenter commented Dec 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.67%. Comparing base (1126cfd) to head (803ea8d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12529      +/-   ##
==========================================
+ Coverage   90.63%   90.67%   +0.03%     
==========================================
  Files         432      434       +2     
  Lines       29730    29755      +25     
==========================================
+ Hits        26947    26981      +34     
+ Misses       2783     2774       -9     
Flag Coverage Δ
cli-tests 37.55% <3.70%> (-0.06%) ⬇️
gui-tests 68.62% <3.70%> (-0.08%) ⬇️
performance-and-unit-tests 74.19% <100.00%> (+0.04%) ⬆️
test 38.28% <3.70%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yngve-sk yngve-sk self-assigned this Dec 19, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Dec 19, 2025

CodSpeed Performance Report

Merging #12529 will not alter performance

Comparing yngve-sk:25.12.remove-summary-refcase-fixup (803ea8d) with main (2981e59)

Summary

✅ 22 untouched

@yngve-sk yngve-sk force-pushed the 25.12.remove-summary-refcase-fixup branch from 71a0e04 to 08ada00 Compare December 19, 2025 13:43
@yngve-sk yngve-sk force-pushed the 25.12.remove-summary-refcase-fixup branch 3 times, most recently from 3bb099a to bf2214b Compare January 5, 2026 06:54
@yngve-sk yngve-sk force-pushed the 25.12.remove-summary-refcase-fixup branch 2 times, most recently from c1946fd to 4947cb6 Compare January 5, 2026 08:44
Remedy bug in removal in to12.py. Added as separate migration, since the to12.py faulty migration is already in effect for older migrated storages.
@yngve-sk yngve-sk force-pushed the 25.12.remove-summary-refcase-fixup branch from 4947cb6 to 803ea8d Compare January 5, 2026 08:50
Copy link
Contributor

@andreas-el andreas-el left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is ready to go in. 🎖️

@github-project-automation github-project-automation bot moved this from Ready for Review to Reviewed in SCOUT Jan 5, 2026
@yngve-sk yngve-sk added the release-notes:breaking-change Automatically categorise as breaking change in release notes label Jan 5, 2026
@yngve-sk yngve-sk merged commit 5e05c3a into equinor:main Jan 5, 2026
36 checks passed
@github-project-automation github-project-automation bot moved this from Reviewed to Done in SCOUT Jan 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes:breaking-change Automatically categorise as breaking change in release notes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants