Skip to content

Conversation

pyrco
Copy link
Contributor

@pyrco pyrco commented Sep 30, 2022

(DIS-962)

@pyrco pyrco force-pushed the feature/dis-962_more-clear-collection-of-non-existent-files branch 2 times, most recently from 5ddba84 to 7bc9deb Compare October 6, 2022 12:18
@codecov
Copy link

codecov bot commented Oct 6, 2022

Codecov Report

Merging #10 (871aafc) into main (0f8c210) will increase coverage by 1.66%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   31.57%   33.23%   +1.66%     
==========================================
  Files          19       19              
  Lines        2084     2091       +7     
==========================================
+ Hits          658      695      +37     
+ Misses       1426     1396      -30     
Flag Coverage Δ
unittests 33.23% <100.00%> (+1.66%) ⬆️

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

Impacted Files Coverage Δ
acquire/collector.py 48.47% <100.00%> (+11.66%) ⬆️

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

@pyrco pyrco force-pushed the feature/dis-962_more-clear-collection-of-non-existent-files branch 4 times, most recently from b18d270 to bf09040 Compare October 10, 2022 09:28
@pyrco
Copy link
Contributor Author

pyrco commented Oct 10, 2022

@Schamper Note: I added calls to reporting when a symlink or an unknown file type are collected (they are now reported as a failed collection).

@pyrco pyrco force-pushed the feature/dis-962_more-clear-collection-of-non-existent-files branch from bf09040 to db22322 Compare October 11, 2022 17:19
Schamper
Schamper previously approved these changes Oct 12, 2022
@pyrco pyrco force-pushed the feature/dis-962_more-clear-collection-of-non-existent-files branch from db22322 to 871aafc Compare October 12, 2022 14:05
@pyrco pyrco merged commit 72441aa into main Oct 13, 2022
@pyrco pyrco deleted the feature/dis-962_more-clear-collection-of-non-existent-files branch October 13, 2022 07:03
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.

2 participants