Fixed CI

[dcavanagh]

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

• [ ] Updated docs / Refactor code / Added a tests case (non-breaking change)
• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• [ ] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [ ] I have read the CONTRIBUTING document.
• [ ] I have added tests to cover my changes.
• [ ] All new and existing tests passed.
• [ ] I have updated the changelog.

[codecov-commenter]

Codecov Report

Merging #1520 (fe804df) into master (0b4a689) will not change coverage. Report is 1 commits behind head on master. The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##           master    #1520   +/-   ##
=======================================
  Coverage   99.56%   99.56%           
=======================================
  Files          53       53           
  Lines        1364     1364           
  Branches      196      196           
=======================================
  Hits         1358     1358           
  Partials        6        6           

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

[Jameskmonger]

Not sure if this is still needed @dcavanagh, CI seems to be working well on all PRs and the main repo.

I see there is some flakiness on the main branch .. is this PR related to that?