Skip to content

Re-initialized ordered list index when no current index available - #528

Merged
amantoux merged 1 commit into
masterfrom
fix/multilist_error
Jul 22, 2025
Merged

Re-initialized ordered list index when no current index available#528
amantoux merged 1 commit into
masterfrom
fix/multilist_error

Conversation

@amantoux

Copy link
Copy Markdown
Member

Fixes #527

@amantoux
amantoux requested a review from Amir-P July 20, 2025 08:11
@codecov

codecov Bot commented Jul 20, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 88.71%. Comparing base (9703426) to head (99d755f).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
.../fleather/lib/src/widgets/editable_text_block.dart 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #528      +/-   ##
==========================================
+ Coverage   88.69%   88.71%   +0.01%     
==========================================
  Files          63       63              
  Lines       10856    10857       +1     
==========================================
+ Hits         9629     9632       +3     
+ Misses       1227     1225       -2     

☔ 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.

@Amir-P Amir-P left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Can you please create an issue to fix the behavior as this only prevents the exception but the underlying issue is still there. @amantoux

@amantoux
amantoux merged commit 5b3a1d7 into master Jul 22, 2025
3 of 4 checks passed
@amantoux
amantoux deleted the fix/multilist_error branch August 2, 2025 15:09
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.

Editing the following specific format will result in an error

2 participants