Skip to content

Conversation

@BugenZhao
Copy link
Member

Signed-off-by: Bugen Zhao [email protected]

BugenZhao added 3 commits June 9, 2025 16:19
Signed-off-by: Bugen Zhao <[email protected]>
Signed-off-by: Bugen Zhao <[email protected]>
@BugenZhao BugenZhao requested a review from Copilot June 9, 2025 08:45
Copy link

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 adds support for attribute macros with the new "attributes" feature. The changes include:

  • Exporting the instrument attribute macro in src/lib.rs when the "attributes" feature is enabled.
  • Adding an example in examples/instrument.rs demonstrating various usages.
  • Including integration and expansion tests as well as updating documentation and Cargo.toml configurations.

Reviewed Changes

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

Show a summary per file
File Description
src/lib.rs Adds conditional export for the instrument macro.
examples/instrument.rs Provides an example demonstrating instrument usage in async functions.
await-tree-attributes/tests/integration.rs Verifies basic usage, visibility, and attributes for the macro.
await-tree-attributes/tests/expansion.rs Ensures the attribute expansion works as expected.
await-tree-attributes/src/lib.rs Implements the instrument attribute macro and argument parsing.
await-tree-attributes/README.md Updates documentation to include usage examples for the attributes feature.
await-tree-attributes/Cargo.toml Configures the crate with necessary dependencies and proc-macro feature.
README.md Documents the new "attributes" feature and provides usage examples.
Cargo.toml Updates workspace configuration to include the await-tree-attributes member and examples.

BugenZhao added 3 commits June 9, 2025 16:46
Signed-off-by: Bugen Zhao <[email protected]>
Signed-off-by: Bugen Zhao <[email protected]>
@BugenZhao BugenZhao enabled auto-merge (squash) June 9, 2025 12:39
@BugenZhao BugenZhao merged commit 6433f7a into main Jun 9, 2025
3 checks passed
@BugenZhao BugenZhao deleted the bz/attribute branch June 9, 2025 12:40
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