Skip to content

Conversation

@Berrysoft
Copy link
Member

@Berrysoft Berrysoft added bug Something isn't working documentation Improvements or additions to documentation labels Nov 3, 2025
@Berrysoft Berrysoft self-assigned this Nov 3, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the doc_auto_cfg feature flag from all library crates in the compio project, leaving only the doc_cfg feature enabled for documentation builds. The change simplifies the documentation configuration by removing automatic feature detection.

  • Removed doc_auto_cfg from the cfg_attr directive across all 11 library crates
  • Retained doc_cfg feature for manual control over documentation of conditional features

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
compio/src/lib.rs Removed doc_auto_cfg from main library crate
compio-tls/src/lib.rs Removed doc_auto_cfg from TLS support crate
compio-signal/src/lib.rs Removed doc_auto_cfg from signal handling crate
compio-runtime/src/lib.rs Removed doc_auto_cfg from runtime crate
compio-quic/src/lib.rs Removed doc_auto_cfg from QUIC protocol crate
compio-process/src/lib.rs Removed doc_auto_cfg from process management crate
compio-net/src/lib.rs Removed doc_auto_cfg from networking crate
compio-io/src/lib.rs Removed doc_auto_cfg from I/O operations crate
compio-fs/src/lib.rs Removed doc_auto_cfg from filesystem operations crate
compio-driver/src/lib.rs Removed doc_auto_cfg from platform driver crate
compio-buf/src/lib.rs Removed doc_auto_cfg from buffer management crate

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@George-Miao George-Miao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Berrysoft Berrysoft merged commit 12f3027 into compio-rs:master Nov 4, 2025
56 checks passed
@Berrysoft Berrysoft deleted the fix/doc_auto_cfg branch November 4, 2025 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants