Replies: 3 comments
-
Also, is there are reason why you use the Enry library instead of linguist? The latter uses ruby which is preinstalled on every Mac. |
Beta Was this translation helpful? Give feedback.
-
I totally agree, language detection is about Markdown editing. this plugin is not a Markdown editor. @relikd I think you should move this title to "Ideas" @sbarex What are your thoughts? |
Beta Was this translation helpful? Give feedback.
-
I added it just for fun. I don't use the .md format much, so I don't even know if there are frequent cases where the language is not specified. Regarding the weight ... unfortunately Apple does not care much about this aspect (otherwise the application that weighs about 63Mb should not shrink to just 17mb when compressed in zip.). As usual the libraries are duplicated to be used both by the app and by the extension ... having the time and inclination should understand if you can avoid this issue. |
Beta Was this translation helpful? Give feedback.
-
Hi, I personally try to reduce dependencies in my projects as much as possible. Ideally 0.
So I would suggest to remove the two dependencies for automatic code language detection.
I would argue that for a quick preview how it would look on Github it is not necessary. I am assuming the main purpose of this app/extension is to check whether the remining formatting is correct, not whether the syntax highlighting of code blocks is correct.
Further, if people really do want correct syntaxt highlighting they can include it in the code block header as such:
The main benefits of removing the dependencies are improved performance and security. In my point of view the added value of automatic language detection is not worth the implications. Also, it would make building from source easier as the users wouldn't need to install go for example.
oh, and the file size would shrink (significantly?) without the deps. What are your thoughts?
Beta Was this translation helpful? Give feedback.
All reactions