iandotkelly / nlf

Node License Finder
MIT License
152 stars 41 forks source link

Matches license when at end of line #38

Closed edi9999 closed 8 years ago

edi9999 commented 8 years ago

When the readme file is :

MIT

(without a trailing new line at the end of the file), the detection of the license fails.

This happens on for example : https://www.npmjs.com/package/object-component, which is a pretty popular package

This fixes this

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 93.355% when pulling 9fa4352594d163befa056cb9c2c76e97ce6c8f10 on edi9999:patch-1 into 27ee316f76b63281435839a6cef548f5786343f6 on iandotkelly:master.

edi9999 commented 8 years ago

The bug failed but I think it is just a flaky test-case : https://travis-ci.org/iandotkelly/nlf/builds/139187631

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 93.355% when pulling 9fa4352594d163befa056cb9c2c76e97ce6c8f10 on edi9999:patch-1 into 27ee316f76b63281435839a6cef548f5786343f6 on iandotkelly:master.

iandotkelly commented 8 years ago

Hi - I'll have a look at this at the weekend - I have a number of things to do and plan to re-release nlf at this time. Thanks for the contribution