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

Indented binutils binary file #685

Merged
merged 1 commit into from
May 22, 2020
Merged

Conversation

kritirikhi
Copy link
Contributor

Fixes #684
This PR fixes the indentation error on binutils binary file

@codecov-commenter
Copy link

Codecov Report

Merging #685 into master will decrease coverage by 7.25%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #685      +/-   ##
==========================================
- Coverage   87.88%   80.62%   -7.26%     
==========================================
  Files          70       70              
  Lines        2204     2204              
  Branches      285      285              
==========================================
- Hits         1937     1777     -160     
- Misses        201      358     +157     
- Partials       66       69       +3     
Flag Coverage Δ
#longtests 80.62% <ø> (-7.26%) ⬇️
Impacted Files Coverage Δ
cve_bin_tool/cvedb.py 52.15% <0.00%> (-30.57%) ⬇️
test/test_json.py 61.76% <0.00%> (-29.42%) ⬇️
cve_bin_tool/checkers/xml2.py 61.36% <0.00%> (-27.28%) ⬇️
test/test_cli.py 82.05% <0.00%> (-17.95%) ⬇️
test/test_scanner.py 83.11% <0.00%> (-9.10%) ⬇️
cve_bin_tool/cli.py 77.58% <0.00%> (-5.52%) ⬇️
cve_bin_tool/extractor.py 61.53% <0.00%> (-2.57%) ⬇️
cve_bin_tool/VersionSignature.py 89.36% <0.00%> (+2.12%) ⬆️

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 51081f0...15ac8a5. Read the comment docs.

@pdxjohnny pdxjohnny merged commit d934bc8 into intel:master May 22, 2020
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.

Bad indentation in test-binutils-2.31.1.c
4 participants