src-d / enry

A faster file programming language detector
https://blog.sourced.tech/post/enry/
Apache License 2.0
460 stars 51 forks source link

Sync to linguist v7.5.1 #234

Closed lafriks closed 5 years ago

lafriks commented 5 years ago

Tests still fail, will check them

lafriks commented 5 years ago

Added special case for osascript interpreter in shebang detection if it is specified with -l argument

(to have same behavior as linguist)

lafriks commented 5 years ago

Will update when dependent PR's are merged

bzz commented 5 years ago

@lafriks thank you for the fixes, all the PRs are merged now!

After rebasing, would you be so kind to also document a new extension-based heuristic failure on samples for .rno at the https://github.com/src-d/enry#divergences-from-linguist?

As soon as this is merged, I'll 🔪 a new release.

lafriks commented 5 years ago

Rebased, I don't think there is new failures except for already documented samples/SQL/drop_stuff.sql. All other tests now should succeed

lafriks commented 5 years ago

ooooh... 😆 I finally got what you did mean, it's about unsupported regexp. Added to docs

lafriks commented 5 years ago

No problem, all Your requests had valid point, I'm also for clean and traceable code ;)