Skip to content

Conversation

johnzl-777
Copy link
Collaborator

@johnzl-777 johnzl-777 commented Jan 12, 2023

  • Instead of using the "hacked together" CI YAML, overhaul to follow example from codecov specifically for Julia (probably more robust and makes things easier to understand)
  • Add codecov badge to README.md
  • Restrict coverage-related steps to only occur on 8-thread test (saves time and 8-thread will also require algorithm to exercise procedure for splitting rows across multiple threads)

@codecov
Copy link

codecov bot commented Jan 12, 2023

Codecov Report

❗ No coverage uploaded for pull request base (main@f96e33a). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head f14fc8b differs from pull request most recent head 0a2c9c9. Consider uploading reports for the commit 0a2c9c9 to get more accurate results

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage        ?   98.55%           
=======================================
  Files           ?        2           
  Lines           ?      138           
  Branches        ?        0           
=======================================
  Hits            ?      136           
  Misses          ?        2           
  Partials        ?        0           

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

@johnzl-777 johnzl-777 changed the title Only Get Coverage on 8-thread test Overhaul Coverage based on Julia Example for Codecov Jan 12, 2023
@johnzl-777 johnzl-777 merged commit f382312 into main Jan 12, 2023
@johnzl-777 johnzl-777 deleted the johnzl-777/get-coverage-only-on-8-threads branch January 12, 2023 16:28
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.

1 participant