Skip to content

Conversation

@lmmx
Copy link
Contributor

@lmmx lmmx commented Nov 23, 2025

  • Discussed in Avoid unused tracing dep attributes feature in syntax/parser crates #21107
  • Avoids activating an attributes feature to crates that do not use it
  • Updates the 6 crates that use attributes feature to specify it in
    their Cargo.toml: {hir,hir-def,hir-ty,ide-assists,ide-db,project-model}
  • Removes the tracing (implicitly dep:tracing) dependency from the parser crate default feature group (which now leaves it empty)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 23, 2025
@lmmx lmmx force-pushed the no-unused-tracing-attributes-feat branch from 76d2f44 to 748feab Compare November 23, 2025 13:26
- Discussed in rust-lang#21107
- Avoids activating an `attributes` feature to crates that do not use it
- Updates the 6 crates that use attributes feature to specify it in
  their Cargo.toml: {hir,hir-def,hir-ty,ide-assists,ide-db,project-model}
@lmmx lmmx force-pushed the no-unused-tracing-attributes-feat branch from 748feab to 2443cce Compare November 23, 2025 13:37
@lmmx
Copy link
Contributor Author

lmmx commented Nov 23, 2025

All checks passing and rebased to trunk 🙂 Ready to merge @Veykril @ChayimFriedman2

Copy link
Contributor

@ChayimFriedman2 ChayimFriedman2 left a comment

Choose a reason for hiding this comment

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

Thanks!

@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue Nov 23, 2025
Merged via the queue into rust-lang:master with commit 9455366 Nov 23, 2025
15 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 23, 2025
@lmmx lmmx deleted the no-unused-tracing-attributes-feat branch November 23, 2025 14: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.

3 participants