Skip to content
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

Issue #5 - Fix for empty projects or code without modules #6

Merged
merged 3 commits into from
Feb 14, 2019

Conversation

matsoftware
Copy link
Owner

This PR fixes the issues related to scenarios where the project doesn't have any module or there are no files by gracefully failing without resulting in a nonzero error.

@matsoftware matsoftware added the bug Something isn't working label Feb 14, 2019
@matsoftware matsoftware added this to the 1.2.3 milestone Feb 14, 2019
@codecov
Copy link

codecov bot commented Feb 14, 2019

Codecov Report

Merging #6 into master will increase coverage by 0.51%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   96.06%   96.58%   +0.51%     
==========================================
  Files           6        6              
  Lines         458      468      +10     
==========================================
+ Hits          440      452      +12     
+ Misses         18       16       -2
Impacted Files Coverage Δ
swift_code_metrics/_analyzer.py 99.27% <100%> (+0.8%) ⬆️
swift_code_metrics/_metrics.py 93.38% <100%> (+0.16%) ⬆️
swift_code_metrics/_helpers.py 100% <0%> (+2.38%) ⬆️

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 9634a52...b5f2562. Read the comment docs.

@matsoftware matsoftware merged commit 8f0b256 into master Feb 14, 2019
@matsoftware matsoftware deleted the issue-n5 branch February 14, 2019 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant