fix: enforce strict clippy lints and eliminate technical debt #37
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
unwrap_used,expect_used,panicclippy lints as deny at workspace levelcfg_attr(test, allow(...))is_emptyparameter)parse_channel()from 104 to 66 lines (extracted helper functions)Changes
Clippy Lints
Added strict lints to
Cargo.toml:Test Exemptions
Test files have proper allow attributes:
#![allow(clippy::unwrap_used, clippy::expect_used, clippy::panic)]Bug Fixes
parse_item_itunes()Code Quality
parse_channel_item()andparse_channel_extension()helpersparse_channel()from 104 to 66 linesTest plan