Skip to content

fix(message_length_limit): align the behavior of message_length_limit - #1813

Merged
Lee-W merged 1 commit into
next-releasefrom
message-length-fix
Jan 31, 2026
Merged

fix(message_length_limit): align the behavior of message_length_limit#1813
Lee-W merged 1 commit into
next-releasefrom
message-length-fix

Conversation

@bearomorphism

@bearomorphismbearomorphism commented Jan 19, 2026

Copy link
Copy Markdown
Collaborator

The document says 0 is no limit

Also, in test_check_command.py, there is no need to copy and paste the function from its base class.

@codecov

codecovBot commented Jan 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.96%. Comparing base (de24815) to head (efd1549).
⚠️ Report is 10 commits behind head on next-release.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@ Coverage Diff @@## next-release #1813 +/- ##
=============================================
Coverage 97.96% 97.96% =============================================
Files 60 60 Lines 2648 2650 +2 =============================================
+ Hits 2594 2596 +2 
Misses 54 54 

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

@Lee-W
Lee-W changed the base branch from master to next-releaseJanuary 31, 2026 06:40
@Lee-W
Lee-W merged commit 15513f7 into next-releaseJan 31, 2026
20 checks passed
@Lee-W
Lee-W deleted the message-length-fix branch January 31, 2026 06:40
Sign up for freeto 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.

2 participants

@bearomorphism@Lee-W