Colin Seymour
Colin Seymour
The link to the source needs to be to the original source, not your fork of Linguist. You've also missed off the line stating the license of the sample.
🤦 I forgot to merge this PR before making the last release.
Whoops. I forgot again. Lets give it a go now.
Of course I need to manually build and push a container image for the current release. I'll try find time this week.
> > You need to add the sample you reference to the PR 😉 > > With all due respect, I do not get why a sample is needed. There...
> 1\. What is the "Pending Popularity" tag? My understanding is that the language is not popular enough for the PR to be merged, is this correct? If so, how...
Looks good. The only thing needed now is for the license to be specified for each sample in the template. We can only accept open-source licensed files and the license...
You don't need to include the license in the PR. We only need them to be stated inline with the links or on the "Sample license(s):" line in the template.