Skip to content

feat(sidekick/rust): remove google_cloud_unstable_tracing guards#5103

Merged
haphungw merged 1 commit intogoogleapis:mainfrom
haphungw:remove-google-cloud-unstable-tracing-guards
Apr 13, 2026
Merged

feat(sidekick/rust): remove google_cloud_unstable_tracing guards#5103
haphungw merged 1 commit intogoogleapis:mainfrom
haphungw:remove-google-cloud-unstable-tracing-guards

Conversation

@haphungw
Copy link
Copy Markdown
Contributor

@haphungw haphungw commented Apr 5, 2026

Remove google_cloud_unstable_tracing conditional compilation guards from the Rust templates. This makes tracing unconditionally available in the generated code, subject to runtime configuration via gaxi::options::tracing_enabled.

This change prepares for the removal of the unstable feature flag in google-cloud-rust: google-cloud-rust/pull/5292

The submission of this PR is blocked on the turbo launch.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the google_cloud_unstable_tracing conditional compilation guards from the Rust templates, transitioning the logic to use Mustache template sections. This change simplifies the generated code by removing compile-time feature flags in favor of template-driven generation. A minor consistency issue was identified regarding an unnecessary space in a closing Mustache tag.

Comment thread internal/sidekick/rust/templates/crate/src/tracing.rs.mustache Outdated
@haphungw haphungw force-pushed the remove-google-cloud-unstable-tracing-guards branch from c729c9f to a56bb53 Compare April 5, 2026 17:18
@haphungw haphungw force-pushed the remove-google-cloud-unstable-tracing-guards branch from a56bb53 to fdf6608 Compare April 13, 2026 12:10
@haphungw haphungw marked this pull request as ready for review April 13, 2026 12:38
@haphungw haphungw requested a review from a team as a code owner April 13, 2026 12:38
@haphungw haphungw merged commit a2a6f2e into googleapis:main Apr 13, 2026
19 of 20 checks passed
haphungw added a commit that referenced this pull request Apr 13, 2026
Remove the redundant braces from the `tracing.rs.mustache` template
googleapis/google-cloud-rust#5292, left behind
during the removal of `google_cloud_unstable_tracing` guards in
#5103.
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