search

github-linguist / linguist

Language Savant. If your repository's language is being reported incorrectly, send us a pull request!
MIT License
12.25k stars 4.23k forks source link

Wrong language for pgloader project #3540

Closed dimitri closed 7 years ago

dimitri commented 7 years ago

Hi,

While linguist seems to do a good job at profiling individual files in the pgloader project as shown at https://github.com/dimitri/pgloader/search?l=plpgsql in the Languages box on the left, it still fails to properly figure out it's mainly a Common Lisp project and shows it as PLpgSQL for some reason: https://github.com/dimitri/pgloader. The stated PLpgSQL 83.6% refers to a single 34 lines long file when clicked...

Can you fix it?

pchaigno commented 7 years ago

The search results are not up-to-date, but the statistics are. I ran Linguist against your repository locally and it found the following SQL files:

test/geolite.sql
test/sqlite/Chinook_Sqlite.sql
test/sqlite/Chinook_Sqlite_AutoIncrementPKs.sql

I'm guessing the two sqlite/ files are quite large and contribute to te 83.6% PLpgSQL

dimitri commented 7 years ago

Those files are test cases initialisation data. How can I tell that to linguist?

pchaigno commented 7 years ago

You can mark those files as vendored using Linguist overrides.

dimitri commented 7 years ago

I failed to install GitHub-linguist locally with brew gem install then brew bundle exec, so I hope the patch is good. What's the usual time to refresh on the GitHub repositories?

dimitri commented 7 years ago

Ok seems it's done already. Thanks!