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

Update logging for automations #10744

Merged
merged 3 commits into from
May 31, 2023
Merged

Update logging for automations #10744

merged 3 commits into from
May 31, 2023

Conversation

PClmnt
Copy link
Collaborator

@PClmnt PClmnt commented May 31, 2023

Description

Add some performance logging within the automation thread.

Copy link
Contributor

@Rory-Powell Rory-Powell left a comment

Choose a reason for hiding this comment

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

It might actually be useful to keep this in! If we format it differently we can pull it out as a metric in dd. Will comment on the line

@PClmnt PClmnt changed the title Remove debug logging introduced in #10737 Update logging for automations May 31, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 31, 2023

Codecov Report

Merging #10744 (050839e) into develop (2d54af3) will decrease coverage by 10.15%.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@             Coverage Diff              @@
##           develop   #10744       +/-   ##
============================================
- Coverage    60.15%   50.00%   -10.15%     
============================================
  Files          245      153       -92     
  Lines         7521     5147     -2374     
  Branches      1405     1029      -376     
============================================
- Hits          4524     2574     -1950     
+ Misses        2757     2357      -400     
+ Partials       240      216       -24     

see 92 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@PClmnt PClmnt merged commit e1ad92c into develop May 31, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 31, 2023
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