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

Skip to content

Conversation

@zlav
Copy link
Member

@zlav zlav commented Jun 10, 2019

Changes made:

  1. We do not currently parse setup.py files and until then we should not be discovering python projects. We can add this method back once we improve python fallbacks.
  2. Don't fail if we don't find the cocoapods binary because we do not actually need it for analysis.

These fixes drive the cli towards a future where we prefer to log warnings instead of failing.

@codecov
Copy link

codecov bot commented Jun 10, 2019

Codecov Report

Merging #470 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #470   +/-   ##
=======================================
  Coverage   52.22%   52.22%           
=======================================
  Files          99       99           
  Lines        5183     5183           
=======================================
  Hits         2707     2707           
  Misses       2212     2212           
  Partials      264      264
Impacted Files Coverage Δ
analyzers/python/python.go 34.9% <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 9f346ef...475e588. Read the comment docs.

@zlav zlav merged commit b7cb71a into master Jun 10, 2019
@zlav zlav deleted the fix/python-cocoapods-fail-easy branch June 10, 2019 22:20
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