Skip to content

Conversation

adamhathcock
Copy link
Member

Adds finalization step for caching to show user more detail.

Fixes progress on transmissions.
Changes Sqlite saving to pre-report progress to allow detection for UI

@adamhathcock adamhathcock requested a review from Copilot August 26, 2025 10:05
Copy link

linear bot commented Aug 26, 2025

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@adamhathcock adamhathcock changed the title Progress the final sqlite message....can still take a while fix(all) Progress the final sqlite message....can still take a while Aug 26, 2025
Copy link

codecov bot commented Aug 26, 2025

Codecov Report

❌ Patch coverage is 95.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.00%. Comparing base (92da66b) to head (953afae).
⚠️ Report is 3 commits behind head on dev.

Files with missing lines Patch % Lines
...eckle.Connectors.Common/Operations/SendProgress.cs 60.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1042      +/-   ##
==========================================
+ Coverage   41.06%   42.00%   +0.93%     
==========================================
  Files         109      109              
  Lines        3090     3119      +29     
  Branches      295      295              
==========================================
+ Hits         1269     1310      +41     
+ Misses       1775     1766       -9     
+ Partials       46       43       -3     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

@bjoernsteinhagen bjoernsteinhagen left a comment

Choose a reason for hiding this comment

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

not gross.

@adamhathcock adamhathcock enabled auto-merge (squash) August 28, 2025 09:45
@adamhathcock adamhathcock merged commit df4c065 into dev Aug 28, 2025
5 checks passed
@adamhathcock adamhathcock deleted the adam/cnx-2408 branch August 28, 2025 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants