Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check group id, metric name, and group name uniqueness #382

Merged
merged 13 commits into from
Nov 22, 2024

feat(check_group): Use itertools unique for clarity.

13d5421
Select commit
Loading
Failed to load commit list.
Merged

Check group id, metric name, and group name uniqueness #382

feat(check_group): Use itertools unique for clarity.
13d5421
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 22, 2024 in 0s

85.0% of diff hit (target 72.5%)

View this Pull Request on Codecov

85.0% of diff hit (target 72.5%)

Annotations

Check warning on line 174 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L174

Added line #L174 was not covered by tests

Check warning on line 176 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L176

Added line #L176 was not covered by tests

Check warning on line 180 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L180

Added line #L180 was not covered by tests

Check warning on line 183 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L183

Added line #L183 was not covered by tests

Check warning on line 193 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L193

Added line #L193 was not covered by tests

Check warning on line 199 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L199

Added line #L199 was not covered by tests

Check warning on line 205 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L205

Added line #L205 was not covered by tests

Check warning on line 223 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L223

Added line #L223 was not covered by tests

Check warning on line 235 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L235

Added line #L235 was not covered by tests

Check warning on line 252 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L252

Added line #L252 was not covered by tests

Check warning on line 281 in crates/weaver_diff/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_diff/src/lib.rs#L281

Added line #L281 was not covered by tests

Check warning on line 136 in crates/weaver_resolver/src/registry.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

crates/weaver_resolver/src/registry.rs#L136

Added line #L136 was not covered by tests