Skip to content

Conversation

@ratmice
Copy link
Collaborator

@ratmice ratmice commented Mar 12, 2025

In adding the rust_edition entry for the cache, it occurred to me we could also simplify the to_variant_tokens impl for Visibility a tiny bit.

}

impl RustEdition {
fn to_variant_tokens(&self) -> TokenStream {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Can just be self because copy.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in 294ce8e

@ltratt
Copy link
Member

ltratt commented Mar 12, 2025

Please squash.

@ratmice ratmice force-pushed the cache_info_rust_edition branch from 294ce8e to 4501e89 Compare March 12, 2025 13:43
@ratmice
Copy link
Collaborator Author

ratmice commented Mar 12, 2025

Squashed.

@ltratt ltratt enabled auto-merge March 12, 2025 13:44
@ltratt ltratt added this pull request to the merge queue Mar 12, 2025
Merged via the queue into softdevteam:master with commit 8022579 Mar 12, 2025
2 checks passed
@ratmice ratmice deleted the cache_info_rust_edition branch March 12, 2025 15:26
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