Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

amandadumi
Copy link
Member

This turns the method filter flag to "true". This completes the carryforward initial implementation, ensuring coverage can remain accurate as we filter tests. Allows the methods tests to only run when a file in the methods folder has changed #1086 , but ensure the previous methods coverage report is carried forward and uploaded to codecov if it hasn't been regenerated for the current pull request ( #1101 and this pull request).

@codecov
Copy link

codecov bot commented Apr 8, 2022

Codecov Report

Merging #1126 (00430ee) into master (7e69cce) will decrease coverage by 8.07%.
The diff coverage is 100.00%.

❗ Current head 00430ee differs from pull request most recent head 4e74c2e. Consider uploading reports for the commit 4e74c2e to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1126      +/-   ##
==========================================
- Coverage   87.85%   79.77%   -8.08%     
==========================================
  Files          69       69              
  Lines       13515    13491      -24     
==========================================
- Hits        11873    10763    -1110     
- Misses       1642     2728    +1086     
Impacted Files Coverage Δ
cclib/parser/orcaparser.py 93.52% <100.00%> (ø)
cclib/method/ddec.py 11.45% <0.00%> (-82.13%) ⬇️
cclib/method/lpa.py 13.04% <0.00%> (-72.47%) ⬇️
cclib/method/bickelhaupt.py 12.50% <0.00%> (-72.23%) ⬇️
cclib/method/cspa.py 16.12% <0.00%> (-70.97%) ⬇️
cclib/method/cda.py 12.67% <0.00%> (-70.43%) ⬇️
cclib/method/mbo.py 12.32% <0.00%> (-69.87%) ⬇️
cclib/method/bader.py 23.25% <0.00%> (-69.77%) ⬇️
cclib/method/population.py 21.81% <0.00%> (-69.10%) ⬇️
cclib/method/mpa.py 14.28% <0.00%> (-66.67%) ⬇️
... and 15 more

@amandadumi amandadumi marked this pull request as draft April 8, 2022 02:46
@amandadumi
Copy link
Member Author

amandadumi commented Apr 8, 2022

This did not work as I expected. I'll look into it!

After a bit of research, it seems like I may have to add something like this: https://docs.codecov.com/docs/commit-status#flag_coverage_not_uploaded_behavior

@berquist berquist added this to the v1.7.2 milestone Apr 16, 2022
@langner langner modified the milestones: v1.7.2, v1.8 May 17, 2022
@shivupa
Copy link
Member

shivupa commented May 30, 2023

What is the status on this?

@amandadumi
Copy link
Member Author

Hi! This is on the back burner until I address #1077 is addressed.

@shivupa shivupa modified the milestones: v1.8, v1.8.1 Jun 13, 2023
@berquist berquist modified the milestones: v1.8.1, v2.0 Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants