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

[OCC] fix undesired colon causing re-executions (#466) #467

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

stevenlanders
Copy link
Contributor

Describe your changes and provide context

  • toExecute is supposed to be overwritten
  • added summary log line

Testing performed to validate your change

  • load tests confirm performance not impacted

## Describe your changes and provide context
- toExecute is supposed to be overwritten 
- added summary log line

## Testing performed to validate your change
- load tests confirm performance not impacted
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.46%. Comparing base (e8a28d3) to head (038f4e7).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #467   +/-   ##
=======================================
  Coverage   55.45%   55.46%           
=======================================
  Files         629      629           
  Lines       53535    53539    +4     
=======================================
+ Hits        29688    29693    +5     
  Misses      21734    21734           
+ Partials     2113     2112    -1     
Files Coverage Δ
tasks/scheduler.go 91.59% <100.00%> (+1.79%) ⬆️

... and 2 files with indirect coverage changes

@stevenlanders stevenlanders merged commit 9ad11f9 into main Mar 19, 2024
13 checks passed
@stevenlanders stevenlanders deleted the remove-excess-colon branch March 19, 2024 21:36
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