Linguist: Rails mime_type.rb considered as "E"?

[siegy22]

Hey!

Why is this file considered as "E"?

See the screenshots how I got this:

On the rails repo:

And by clicking the "E":

[Alhadis]

That would be because of this line, here:

# -*- frozen-string-literal: true -*-

The current Emacs modeline is a little buggy and needs fixing... I'm currently working on that, actually.

You can fix this issue by moving the modeline to the bottom of the file, as Linguist only scans the first five lines to look for a modeline.

[siegy22]

Okay, but when I replace it with # frozen_string_literal: true it should work right?

[Alhadis]

Yes. It's the -*- parts that're telling Linguist it's an Emacs modeline.

[arthurnn]

This seems like a bug indeed.

[Alhadis]

Don't worry, I'm revising the modeline regex as we speak. :)

[Alhadis]

@arthurnn Done. #3227 will fix this as well as a similar issue reported previously.

[Alhadis]

@pchaigno, this can be closed, because everything's fixed. =)

Note the refreshing absence of "E" in the language breakdown.

[pchaigno]

Thanks @siegy22 for reporting the issue! Thanks @Alhadis for fixing it :+1: