Colin Seymour

Yes. That's removing the biggest contributors so we can get a better idea of overall usage as GitHub's search is struggling at the moment. See https://github.com/github/linguist/issues/5756 for the temporary criteria...

I'll start with this question... > What would be the minimum number of users and code files required to accept the PR ? The guideline says 200 user/repo The 200...

> Maybe it's worth finding something more up-to-date. As you are familiar with this language and the expected syntax highlighting, you're in the best position to find a more apt...

I'm going to hold off looking into this until the new year. As you stated, this is quite a big change which I want to thoroughly review, test and run...

I've started playing with the changes in this PR and from a performance perspective, this is looking pretty impressive and seems to be about 3-4 times quicker on my MBP...

> Running the [same sample](https://github.com/github/linguist/files/251768/ChatKeys.zip) through the Centroid-based classifier yields the same results as our current Bayesian classifier: It doesn't if you use the full [staging branch](https://github.com/smola/linguist/tree/staging) which pulls in...

The test failure here is because our bogus file with invalid unicode chars at https://github.com/github/linguist/blob/test/attributes/test/fixtures/ba%EF%BF%BDr/file_%C3%A3.pl is now being detected as Raku instead of Perl. That's not really an issue as...

I've gone ahead and pushed the test fix so I can update this PR and start testing it.

> I'm kind of surprised it isn't already detected. That'll be because no one has submitted a PR to add support 😉 . If the usage of either or both...

Peeps normally use the same grammar they use in Textmate, Atom or VSCode. They're all compatible with Linguist, so if you use any of those editors, you could use the...