Skip to content

Conversation

@joaomcteixeira
Copy link
Member

Since this repository is already activated in codecov, we can add the codecov report from the github actions. I forgot to check this on #110 .

@joaomcteixeira joaomcteixeira added the CI / repository Continuous integration or repository organization label Nov 10, 2021
@joaomcteixeira joaomcteixeira added this to the v3.0.0 stable release milestone Nov 10, 2021
@joaomcteixeira joaomcteixeira self-assigned this Nov 10, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 10, 2021

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage        ?   43.73%           
=======================================
  Files           ?       38           
  Lines           ?     1891           
  Branches        ?        0           
=======================================
  Hits            ?      827           
  Misses          ?     1064           
  Partials        ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c460ca4...c706967. Read the comment docs.

@joaomcteixeira
Copy link
Member Author

Conclusion, there's a lot for work to do on testing yet ☺️

Copy link
Member

@amjjbonvin amjjbonvin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess good practices - green light on my side

@joaomcteixeira joaomcteixeira merged commit 6c802df into haddocking:main Nov 11, 2021
@joaomcteixeira joaomcteixeira deleted the codecoving branch November 11, 2021 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI / repository Continuous integration or repository organization

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants