Skip to content

fix(giga): three fixes for v2-matching correctness#2889

Merged
arajasek merged 3 commits intomainfrom
asr/staging-fixes-into-main
Feb 18, 2026
Merged

fix(giga): three fixes for v2-matching correctness#2889
arajasek merged 3 commits intomainfrom
asr/staging-fixes-into-main

Conversation

@arajasek
Copy link
Contributor

@arajasek arajasek commented Feb 13, 2026

Describe your changes and provide context

Three commits:

Testing performed to validate your change

@github-actions
Copy link

github-actions bot commented Feb 13, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 18, 2026, 3:50 PM

@github-actions
Copy link

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 13, 2026, 3:26 PM

@arajasek arajasek changed the title Asr/staging fixes into main fix(giga): four fixes for v2-matching correctness Feb 13, 2026
@arajasek arajasek force-pushed the asr/staging-fixes-into-main branch from 800bd27 to 15ce6be Compare February 13, 2026 15:28
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 33.33333% with 40 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.31%. Comparing base (2a1ccca) to head (39b8c23).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
app/app.go 29.62% 32 Missing and 6 partials ⚠️
giga/executor/executor.go 66.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2889      +/-   ##
==========================================
- Coverage   57.33%   57.31%   -0.02%     
==========================================
  Files        2089     2089              
  Lines      172126   172181      +55     
==========================================
+ Hits        98683    98687       +4     
- Misses      64646    64691      +45     
- Partials     8797     8803       +6     
Flag Coverage Δ
sei-chain 52.79% <33.33%> (-0.02%) ⬇️
sei-cosmos 48.19% <ø> (ø)
sei-db 68.72% <ø> (ø)

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

Files with missing lines Coverage Δ
giga/executor/executor.go 32.00% <66.66%> (-10.11%) ⬇️
app/app.go 71.10% <29.62%> (-1.65%) ⬇️

... and 25 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@arajasek arajasek force-pushed the asr/staging-fixes-into-main branch 2 times, most recently from f655732 to add8a7d Compare February 13, 2026 18:21
@arajasek
Copy link
Contributor Author

Update: dropped the port of #2832 since it needs further discussion.

@arajasek arajasek changed the title fix(giga): four fixes for v2-matching correctness fix(giga): three fixes for v2-matching correctness Feb 13, 2026
@arajasek arajasek force-pushed the asr/staging-fixes-into-main branch from add8a7d to a7a7753 Compare February 17, 2026 14:28
@arajasek arajasek force-pushed the asr/staging-fixes-into-main branch from a7a7753 to 1d1eb11 Compare February 18, 2026 14:41
@arajasek arajasek enabled auto-merge (squash) February 18, 2026 14:41
@arajasek arajasek force-pushed the asr/staging-fixes-into-main branch from 1d1eb11 to c846822 Compare February 18, 2026 14:57
@arajasek arajasek force-pushed the asr/staging-fixes-into-main branch from c846822 to 39b8c23 Compare February 18, 2026 15:49
@arajasek arajasek merged commit 0d430cb into main Feb 18, 2026
41 of 45 checks passed
@arajasek arajasek deleted the asr/staging-fixes-into-main branch February 18, 2026 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments