You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am considering adding support for bitbake, however some of the file extensions used by bitbake are quite ambiguous, such as *.class, *.inc, which might collide with other languages added in the future. As such I propose adding support for declaring the language within a comment within the scanned file, e.g.: # todockeck-lang=bitbake for such cases.
Is this something you would feel comfortable with @preslavmihaylov?
The text was updated successfully, but these errors were encountered:
I think this configuration knob will be a bit cumbersome to use, especially if you have to put it in every file you have.
Alternatively, I think what is more appropriate is to add an "auto-detect lang" feature - instead of determining the language based on extension, todocheck can detect it based on its syntax.
There ought to be some libs out there which can do the job.
If you're up to tinkering with this, I am happy to add it into the tool as a fallback mechanism to determine language instead of using extension.
Alternatively, I think what is more appropriate is to add an "auto-detect lang" feature - instead of determining the language based on extension, todocheck can detect it based on its syntax.
Mhm, I am not thrilled with the idea. IMO auto-discovery would be way to complex and error prone.
There ought to be some libs out there which can do the job.
At least for our use-case there probably isn't. Bitbake is a bit too niche for that.
I have an alternative proposal:
How about an option of setting the language for a specific extension within the projects todocheck config? So rather than on a per-file basis on a per-project basis. That would be less cumbersome while avoiding the complexity of syntax detection.
Later on support for overriding project-wide settings on a per-file basis could be added. This would also be useful for supporting files that do not have an extension, e.g. Dockerfile or Dockerfile-testing.
👋
I am considering adding support for bitbake, however some of the file extensions used by bitbake are quite ambiguous, such as
*.class
,*.inc
, which might collide with other languages added in the future. As such I propose adding support for declaring the language within a comment within the scanned file, e.g.:# todockeck-lang=bitbake
for such cases.Is this something you would feel comfortable with @preslavmihaylov?
The text was updated successfully, but these errors were encountered: