Skip to content

Improve documentation with fixes, comments, and examples#636

Merged
Serial-ATA merged 1 commit intoSerial-ATA:mainfrom
thedavidweng:docs/improve-documentation
Apr 12, 2026
Merged

Improve documentation with fixes, comments, and examples#636
Serial-ATA merged 1 commit intoSerial-ATA:mainfrom
thedavidweng:docs/improve-documentation

Conversation

@thedavidweng
Copy link
Copy Markdown
Contributor

Typos & stale references:
Fixed grammar in doc/ISSUES.md and doc/PULL_REQUESTS.md
Fixed broken file path references in doc/NEW_FILE.md and doc/NEW_TAG.md (6 stale ../src/ paths → ../lofty/src/)
Fixed code example bugs in doc/NEW_TAG.md (missing &mut self, wrong return type)
Updated fuzz example imports in doc/NEW_FILE.md
Fixed typo in lofty/src/tag/item.rs

Removed ALL #[allow(missing_docs)] (previously 15+ files):
Added doc comments to ~200 items across: FileType, MpegVersion, Layer, ChannelMode, Emphasis, WavFormat, PictureType, Mp4Codec, AudioObjectType, ChannelType, EventType, SyncTextContentType, FrameHeader, TimestampFrame, Timestamp, ReplayGain, EncoderInfo, SplitTagRemainder, ItemKey (~88 variants), ChannelMask (18 constants)

Examples:
Added doc-tested examples to Timestamp and TagItem structs

Module docs:
Added/improved //! module docs for ape/tag, id3/v2/tag, iff, id3

@Serial-ATA Serial-ATA force-pushed the docs/improve-documentation branch from 6a61696 to 9747266 Compare April 12, 2026 15:50
Co-authored-by: Serial <69764315+serial-ata@users.noreply.github.com>
@Serial-ATA Serial-ATA force-pushed the docs/improve-documentation branch from 9747266 to 3a3a829 Compare April 12, 2026 15:51
@Serial-ATA Serial-ATA added this to the 0.24.0 milestone Apr 12, 2026
@Serial-ATA
Copy link
Copy Markdown
Owner

Thanks! Did a bit of cleanup and added some links

@Serial-ATA Serial-ATA merged commit 4a2bcb4 into Serial-ATA:main Apr 12, 2026
4 checks passed
@thedavidweng thedavidweng deleted the docs/improve-documentation branch April 13, 2026 07:58
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.

2 participants