Skip to content

Clarify string type in namespace#212

Merged
nmattia merged 1 commit into
mainfrom
nm-old-rust
Jun 27, 2026
Merged

Clarify string type in namespace#212
nmattia merged 1 commit into
mainfrom
nm-old-rust

Conversation

@nmattia

@nmattia nmattia commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

This helps older (1.93) rust versions infer the correct types and works around the following error:

expected &[&String], found &[&str; 2]

This helps older (1.93) rust versions infer the correct types and works
around the following error:

> expected &[&String], found &[&str; 2]
@nmattia nmattia requested a review from a team as a code owner June 26, 2026 20:15
@zeropath-ai

zeropath-ai Bot commented Jun 26, 2026

Copy link
Copy Markdown

No security or compliance issues detected. Reviewed everything up to c8222fa.

Security Overview
Detected Code Changes
Change Type Relevant files
Refactor ► src/routing/ic/route_provider/health.rs
    Update label values for metrics to use as_str()
► src/routing/ic/routing_table_manager.rs
    Update label values for metrics to use as_str()

@nmattia nmattia enabled auto-merge June 26, 2026 20:23
@nmattia nmattia merged commit 3e69897 into main Jun 27, 2026
6 checks passed
@nmattia nmattia deleted the nm-old-rust branch June 27, 2026 11:12
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