Skip to content

Conversation

@wmmc88
Copy link
Collaborator

@wmmc88 wmmc88 commented Aug 4, 2025

This pull request focuses on improving documentation consistency and clarity across multiple crates in the project. The main updates involve standardizing crate references in documentation comments, correcting links to use the appropriate Rust or HTML formats, and clarifying crate names for better readability and navigation.

Documentation consistency and crate references:

  • Standardized documentation comments in crates/cargo-wdk/src/main.rs and crates/wdk-build/src/lib.rs to refer to the crate using the [crate] link format, improving consistency and navigation for users. [1] [2] [3] [4]
  • Updated references to wdk-sys in documentation comments to use the correct wdk_sys naming and HTML link format in crates/wdk-macros/src/lib.rs and crates/wdk/src/lib.rs, ensuring accurate and clickable documentation links. [1] [2] [3]

@wmmc88 wmmc88 requested review from Copilot and leon-xd and removed request for Copilot August 4, 2025 23:29
leon-xd
leon-xd previously approved these changes Aug 4, 2025
Copy link
Contributor

@leon-xd leon-xd left a comment

Choose a reason for hiding this comment

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

thanks for getting this done!

Copilot AI review requested due to automatic review settings August 4, 2025 23:35
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 pull request standardizes documentation references across multiple crates in the Windows Driver Kit (WDK) project. The changes focus on improving consistency in how crate names are referenced in documentation comments and ensuring proper linking formats.

Key changes include:

  • Converting hardcoded crate names to use Rust's [crate] link format for better navigation
  • Standardizing wdk-sys references to use the correct wdk_sys naming convention with HTML links
  • Adding explicit crate context to documentation for clarity

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
crates/wdk/src/lib.rs Updated wdk-sys reference to use correct wdk_sys naming
crates/wdk-macros/src/lib.rs Converted wdk-sys references to proper wdk_sys HTML links
crates/wdk-build/src/lib.rs Replaced hardcoded crate names with [crate] link format
crates/cargo-wdk/src/main.rs Replaced hardcoded crate names with [crate] link format

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Melvin Wang <[email protected]>
@wmmc88 wmmc88 self-assigned this Aug 5, 2025
@wmmc88 wmmc88 enabled auto-merge August 5, 2025 00:25
@wmmc88 wmmc88 added this pull request to the merge queue Aug 5, 2025
Merged via the queue into microsoft:main with commit 0059729 Aug 5, 2025
63 checks passed
@wmmc88 wmmc88 deleted the fix-nightly-doc branch August 5, 2025 00:31
@leon-xd leon-xd mentioned this pull request Oct 17, 2025
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.

3 participants