Skip to content

Conversation

gnikit
Copy link
Member

@gnikit gnikit commented Aug 19, 2022

Hitting the timeout in mocha unittesting leads to c8
creating erroneous coverage reports

Fixes #613

@gnikit
Copy link
Member Author

gnikit commented Aug 19, 2022

@fortran-lang/admins I have send an access request to enable codecov, do you think we can give it a shot.

@awvwgk
Copy link
Member

awvwgk commented Aug 19, 2022

codecov is enabled in Fortran-lang already, I added this repository to the list of repos it is available on.

@gnikit
Copy link
Member Author

gnikit commented Aug 19, 2022

codecov is enabled in Fortran-lang already, I added this repository to the list of repos it is available on.

Cheers @awvwgk

@awvwgk
Copy link
Member

awvwgk commented Aug 19, 2022

The request is for private repo access, do we need this for @fortran-lang?

@gnikit
Copy link
Member Author

gnikit commented Aug 19, 2022

The request is for private repo access, do we need this for @fortran-lang?

nope, I think that was a mistake from my end.

@gnikit
Copy link
Member Author

gnikit commented Aug 19, 2022

@awvwgk Can you see vscode-fortran-support on your codecov list of repos? I can't seen fortran-lang there

image

I think the request was for private repos + modifying public repos which I guess includes creating and uploading cov stats

@awvwgk
Copy link
Member

awvwgk commented Aug 19, 2022

I think codecov is safe to have installed by default for the whole organization rather than only a dedicated repo list. Also approved the request for additional permissions (we don't have any private repos in the org anyway).

@codecov
Copy link

codecov bot commented Aug 21, 2022

Codecov Report

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

❗ Current head 597469f differs from pull request most recent head b862706. Consider uploading reports for the commit b862706 to get more accurate results

@@           Coverage Diff           @@
##             main     #629   +/-   ##
=======================================
  Coverage        ?   68.28%           
=======================================
  Files           ?       11           
  Lines           ?     2103           
  Branches        ?      203           
=======================================
  Hits            ?     1436           
  Misses          ?      665           
  Partials        ?        2           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

gnikit added 2 commits August 21, 2022 15:42
Fixes #613

Hitting mocha timeout creates false coverage reports,
that leads to `c8` reporting false statistics.
@gnikit gnikit merged commit 4b56dc6 into main Aug 21, 2022
@gnikit gnikit deleted the gnikit/issue613 branch August 21, 2022 14:44
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.

feat: enable code coverage
2 participants