Colin Seymour

Results 237 comments of Colin Seymour

> [29](https://github.com/github/linguist/runs/5140459955?check_suite_focus=true#step:5:29) 6502 Assembly/tennis.inc BAD (HTML) > [30](https://github.com/github/linguist/runs/5140459955?check_suite_focus=true#step:5:30) 6502 Assembly/vera.inc BAD (PHP) Both of these appear to be because quite simple and look like variable assignments in other languages and...

When you've got a mo @Alhadis, I'd appreciate your 👀 on the regexes here. 🙇

> I would very much appreciate some guidance on next steps. Thank you. Sure... > Here's [my fork](https://github.com/familyfriendlymikey/linguist/commit/ce6fb293dcbe04b702f4f347dd7ba83143f66c35), grammar compilation failed citing several PCRE errors, despite syntax highlighting working in...

I've been thinking about this and the one thing that's kinda niggling to me is the fact we've got to keep updating the `candidates.json` file which is going to involve...

Yeah, I was thinking that maybe parsing a value from the template or a comment would do the trick. We can always go back and add it to already open...

None of those extensions look popular enough yet either. Each extension on it own needs to meet the minimum usage level for inclusion.

A PR already exists but usage wasn't sufficient at the time https://github.com/github/linguist/pull/4645

> Since Environment Variables are something special, they could be treated as a new language Env or just ignore it. 🤔 I'm not sure that's really the right thing to...

They do indeed read them but they almost always use them as they would environment variables. The file is also called `.env` suggesting it contains environment variables which are technically...

> an .env file has nothing to do with bourne shell scripts. It's just a text configuration file format. It may be in your usage however it's way more than...