New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: log info if a file shows up as a directory #430
Conversation
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
3 similar comments
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
Test Failures Detected: Due to failing tests, we cannot provide coverage reports at this time. ❌ Failed Test Results:Completed 2664 tests with View the full list of failed tests
|
@@ -139,6 +139,8 @@ def _get_file_fixes( | |||
reason=err.reason, | |||
), | |||
) | |||
except IsADirectoryError as err: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the relevant section
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
3 similar comments
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
305474c
to
f01b2e7
Compare
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
3 similar comments
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
3 similar comments
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 666 tests with View the full list of failed tests
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't hurt to have a test for it, but LGTM
fixes codecov/codecov-action#1303
git ls-files
will pull submodules as files, this change just logs that a file shows up as such a directory and continues on