Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

use InterpolatedStringHandler to move values to CustomDimensions Tags instead of keeping them in the error message #2450

Merged
merged 3 commits into from
Sep 27, 2022

Conversation

stishkin
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 27, 2022

Codecov Report

Merging #2450 (1c68ce4) into main (b3748e4) will increase coverage by 0.03%.
The diff coverage is 7.29%.

@@            Coverage Diff             @@
##             main    #2450      +/-   ##
==========================================
+ Coverage   29.85%   29.89%   +0.03%     
==========================================
  Files         288      288              
  Lines       35156    35175      +19     
==========================================
+ Hits        10497    10515      +18     
- Misses      24659    24660       +1     
Impacted Files Coverage Δ
...piService/ApiService/Functions/AgentCanSchedule.cs 0.00% <0.00%> (ø)
...c/ApiService/ApiService/Functions/AgentCommands.cs 0.00% <0.00%> (ø)
.../ApiService/ApiService/Functions/InstanceConfig.cs 0.00% <0.00%> (ø)
src/ApiService/ApiService/Functions/Jobs.cs 76.59% <0.00%> (ø)
src/ApiService/ApiService/Functions/Node.cs 79.56% <0.00%> (ø)
...c/ApiService/ApiService/Functions/Notifications.cs 0.00% <0.00%> (ø)
src/ApiService/ApiService/Functions/Proxy.cs 0.00% <0.00%> (ø)
...piService/ApiService/Functions/QueueFileChanges.cs 0.00% <0.00%> (ø)
...Service/ApiService/Functions/QueueNodeHeartbeat.cs 0.00% <0.00%> (ø)
...piService/ApiService/Functions/QueueProxyUpdate.cs 0.00% <0.00%> (ø)
... and 67 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@stishkin stishkin merged commit 476c99a into main Sep 27, 2022
@stishkin stishkin deleted the stas/logs branch September 27, 2022 22:22
@ghost ghost locked as resolved and limited conversation to collaborators Oct 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants