Rename encoded_reader to raw_content - #3208
Conversation
There was a problem hiding this comment.
Code Review
This pull request renames the encoded_reader method to raw_content on both SolidEntry and NormalEntry structs in lib/src/entry.rs. The old encoded_reader method is retained as a deprecated alias to maintain backwards compatibility. Internal usages and test cases have been updated to use the new raw_content method. There are no review comments to address, so I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
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 ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Closing: keeping |
Summary
Rename
encoded_readertoraw_contentonNormalEntryandSolidEntry, aligning the raw-bytes accessor with thecontent()vocabulary introduced in #3207. The old name remains as a deprecated delegating alias.Notes
#[deprecated(since = "0.36.0", note = "use raw_content() instead")]— behavior is identical (returns the encoded chunk body bytes before decryption/decompression).feat/entry-content); retarget tomainafter it merges.