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.

❗ 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           

📣 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?