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

feat(checker): Change default metric name to T1 #548

Merged
merged 1 commit into from
Jun 18, 2020

Conversation

litleleprikon
Copy link
Contributor

If all targets are marked as targets with alone metrics there is a
inconvenient which metric name should be used in check result. This
commit change default metric name from first alphabetical metric name to
metric name from T1.

Relates #428

@litleleprikon litleleprikon requested a review from a team as a code owner June 16, 2020 11:02
@coveralls
Copy link

coveralls commented Jun 16, 2020

Coverage Status

Coverage decreased (-0.01%) to 80.838% when pulling 2358303 on feature/use-metric-name-from-t1 into 74ecfe9 on master.

If all targets are marked as targets with alone metrics there is a
inconvenient which metric name should be used in check result. This
commit change default metric name from first alphabetical metric name to
metric name from T1.

Relates #428
@litleleprikon litleleprikon force-pushed the feature/use-metric-name-from-t1 branch from 2dbae6d to 2358303 Compare June 18, 2020 11:20
@litleleprikon litleleprikon merged commit 6225534 into master Jun 18, 2020
@litleleprikon litleleprikon deleted the feature/use-metric-name-from-t1 branch June 18, 2020 11:50
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