Fix formatting issues after failed merge#91
Conversation
Reviewer's GuideRefactored the LengthFormat API by extracting constructors and a dedicated Class diagram for updated LengthFormat APIclassDiagram
class LengthFormat {
+usize bytes
+Endianness endianness
+const fn new(bytes: usize, endianness: Endianness) -> Self
+const fn u16_be() -> Self
+const fn u16_le() -> Self
+const fn u32_be() -> Self
+const fn u32_le() -> Self
+fn read_len(&self, bytes: &[u8]) -> io::Result<usize>
}
class Endianness {
<<enum>>
Big
Little
}
LengthFormat --> Endianness
Class diagram for default implementation of WireframeAppclassDiagram
class WireframeApp {
+routes: HashMap
+services
+middleware
+app_data
+serializer
+fn default() -> Self
}
WireframeApp : default() sets empty routes, services, middleware, app_data
WireframeApp : default() sets default serializer
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate Unit Tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Code Health Improved
(1 files improve in Code Health)
Gates Passed
6 Quality Gates Passed
See analysis details in CodeScene
View Improvements
| File | Code Health Impact | Categories Improved |
|---|---|---|
| response.rs | 9.39 → 10.00 | Code Duplication |
Quality Gate Profile: Pay Down Tech Debt
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.
63995ff
into
codex/define-endianness-enum-and-refactor-lengthprefixedprocessor
* Add configurable length format for frames * 📝 Add docstrings to `codex/define-endianness-enum-and-refactor-lengthprefixedprocessor` (#89) * 📝 Add docstrings to `codex/define-endianness-enum-and-refactor-lengthprefixedprocessor` Docstrings generation was requested by @leynos. * #87 (comment) The following files were modified: * `src/app.rs` * `src/frame.rs` * `tests/response.rs` * Move attributes below doc comments (#90) --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Leynos <leynos@troubledskies.net> * Refine length prefix handling * Use rstest for length format tests * Fix syntax errors and update tests (#91) --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Summary
WireframeAppdefault implementationLengthFormatutilitiesTesting
cargo fmt --all -- --checkmake lintmake testhttps://chatgpt.com/codex/tasks/task_e_685581d8a5548322a9d13c55037c6e15
Summary by Sourcery
Restore and enhance length framing utilities, correct application default formatting, and revamp response framing tests with rstest parameterization and new error handling checks.
Enhancements:
Tests: