heat icon indicating copy to clipboard operation
heat copied to clipboard

Update ci.yaml

Open mtar opened this issue 2 years ago • 2 comments

Description

Issue/s resolved: #980

Changes proposed:

  • Wait for precommit

Type of change

Memory requirements

Performance

Due Diligence

  • [ ] All split configurations tested
  • [ ] Multiple dtypes tested in relevant functions
  • [ ] Documentation updated (if needed)
  • [ ] Updated changelog.md under the title "Pending Additions"

Does this change modify the behaviour of other functions? If so, which?

yes / no

skip ci

mtar avatar Jun 08 '22 08:06 mtar

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

ghost avatar Jun 08 '22 08:06 ghost

Codecov Report

Merging #981 (b6290ab) into main (cbce1b9) will not change coverage. The diff coverage is n/a.

:exclamation: Current head b6290ab differs from pull request most recent head 8492e58. Consider uploading reports for the commit 8492e58 to get more accurate results

@@           Coverage Diff           @@
##             main     #981   +/-   ##
=======================================
  Coverage   91.68%   91.68%           
=======================================
  Files          65       65           
  Lines        9978     9978           
=======================================
  Hits         9148     9148           
  Misses        830      830           
Flag Coverage Δ
unit 91.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

codecov[bot] avatar Jun 08 '22 09:06 codecov[bot]