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

Fixing Issue #200: packages with bin that haven't been detected #358

Merged
merged 1 commit into from
Jun 17, 2019

Conversation

2roy999
Copy link
Contributor

@2roy999 2roy999 commented Jun 17, 2019

Some packages have a string of a single binary file instead of an object with the names of the binaries. This caused depcheck to declare those packages as missed. (Issue #200)

@rumpl
Copy link
Member

rumpl commented Jun 17, 2019

Thanks for making this PR @2roy999 ! Code looks good, will merge it as soon as the build is green.

Do you need this quickly or can I wait a little to publish a new version?

@codecov
Copy link

codecov bot commented Jun 17, 2019

Codecov Report

Merging #358 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #358      +/-   ##
==========================================
+ Coverage   98.83%   98.83%   +<.01%     
==========================================
  Files          34       34              
  Lines         684      687       +3     
==========================================
+ Hits          676      679       +3     
  Misses          8        8
Impacted Files Coverage Δ
src/special/bin.js 100% <100%> (ø) ⬆️

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 13bd6e8...68820b6. Read the comment docs.

@2roy999
Copy link
Contributor Author

2roy999 commented Jun 17, 2019

@rumpl It can wait for the next version. Thank you :)

@rumpl rumpl merged commit 55a729d into depcheck:master Jun 17, 2019
@rumpl
Copy link
Member

rumpl commented Jun 17, 2019

Thanks @2roy999 !

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.

None yet

2 participants