Colin Seymour
Colin Seymour
Any sample added to a PR needs to be locatable on GitHub with a clear license covering the source.
> > Please can you replace your new samples which are not real-world files with real-world examples. > > Do you mean the CASM ones ([1](https://github.com/github-linguist/linguist/blob/066b07b201e9ac1b287ed0560de57f406e1541a3/samples/Cairo%200/fib.cairo), [2](https://github.com/github-linguist/linguist/blob/066b07b201e9ac1b287ed0560de57f406e1541a3/samples/Cairo%200/fib_u128.cairo))? Yes, they are...
I make a release approx every 3-4 months. The last one was on 7 Jun so it won't be for a few months... probably late August.
As an FYI, `dune-project` currently meets our usage requirements. `dune` is close. The other two are miles off. In short, everything but `dune-project` is preventing this PR from being merged....
This PR will be held back until _all_ of the extensions you are adding each meet our [usage requirements](https://github.com/github-linguist/linguist/issues/5756). Rather than keeping this hanging around potentially forever, I recommend adding...
There's no tooling but https://github.com/github-linguist/linguist/issues/5756 gives you an idea of how we currently assess popularity.
I wouldn't include the keyword unless this is the only way to identify BAML files as this cuts down quite a bit on the search results. As it appears a...
> Regarding the popularity criteria, isn't it 200 repositories enough for a file extension to be included? No. See https://github.com/github-linguist/linguist/issues/5756 which is referenced in the CONTRIBUTING.md file too.
> Indeed, unlike other assets associated to that language, the nature of `.wlt` files is to have a single one per project which determines how the editor UI is show...
Linguist already supports CSV… it just doesn't have a grammar, hence no syntax highlighting: https://github.com/github-linguist/linguist/blob/e82dba118ba3d3ee715f95d5aada4f14a6f784ac/lib/linguist/languages.yml#L982-L989 A better title for this PR is something like "Grammar required for CSV for syntax...