Skip to content

Conversation

@freitagbr
Copy link
Contributor

Fixes #733 by specifically checking if the stat'ed file is a symlink and that the linked file is a directory.

@freitagbr freitagbr added the fix Bug/defect, or a fix for such a problem label Aug 23, 2017
@freitagbr freitagbr added this to the v0.8.0 milestone Aug 23, 2017
@freitagbr freitagbr requested a review from nfischer August 23, 2017 07:36
@codecov-io
Copy link

Codecov Report

Merging #764 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #764      +/-   ##
==========================================
+ Coverage   97.38%   97.39%   +0.01%     
==========================================
  Files          33       33              
  Lines        1222     1227       +5     
==========================================
+ Hits         1190     1195       +5     
  Misses         32       32
Impacted Files Coverage Δ
src/ls.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 2ee83eb...aa78007. Read the comment docs.

@freitagbr freitagbr merged commit c7d65ac into master Aug 24, 2017
@freitagbr freitagbr deleted the fix-ls-symlink-to-dir branch August 24, 2017 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug/defect, or a fix for such a problem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants