<p><b>@b4n</b> requested changes on this pull request.</p>

<p>Looks reasonable, but do we want Nim builtin?  I don't mind, but for not very widespread languages that only need/have a custom filetype, we often just have a <a href="https://wiki.geany.org/tag/filetype?do=showtag&tag=filetype" rel="nofollow">wiki page</a>.  However, if the filetype is of sufficient quality I guess we can have it in, especially as I read a few mentions of Nim lately here and at Universal-CTags so it might be gaining popularity.</p><hr>

<p>In <a href="https://github.com/geany/geany/pull/2085#discussion_r255291150">data/filedefs/filetypes.Nim.conf</a>:</p>
<pre style='color:#555'>> +
+primary=addr as asm atomic bind block break case cast const continue converter defer discard distinct div do elif else end enum except export finally for from func generic if import include interface iterator let macro method mixin mod notin object of out proc raise return shl shr static template try type using var when while with without yield seq array tuple ref ptr not and xor in or is isnot new result
+
+identifiers=bool float int string nil true false echo
+
+[settings]
+
+extension=nim
+comment_single=#
+comment_use_indent=true
+tag_parser=Python
+lexer_filetype=Python
+
+[lexer_properties]
+styling.within.preprocessor=1
+lexer.cpp.track.preprocessor=0
</pre>
<p>Those two seem irrelevant in case of the Python lexer, aren't they?</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2085#discussion_r255291167">data/filedefs/filetypes.Nim.conf</a>:</p>
<pre style='color:#555'>> +[settings]
+
+extension=nim
+comment_single=#
+comment_use_indent=true
+tag_parser=Python
+lexer_filetype=Python
+
+[lexer_properties]
+styling.within.preprocessor=1
+lexer.cpp.track.preprocessor=0
+
+[indentation]
+
+width=2
+type=0
</pre>
<p>Is this required by the NIM language?  We prefer to not force indent type on people unless it's a requirement of the language (e.g. tabs for Makefiles).  If it's the official recommendation, it's fine to provide those commented-out.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2085#discussion_r255291337">data/filedefs/filetypes.Nim.conf</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,34 @@
+# For complete documentation of this file, please see Geany's main documentation
</pre>
<p>This file needs to be added to <em>data/Makefile.am</em></p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2085#pullrequestreview-201855022">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ3MPIqJl0YhtWHIWy2UTFPUDF8flks5vLoNXgaJpZM4anw0r">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ4xjs_qoUpxDdLEfe-gM_3Y8g7ITks5vLoNXgaJpZM4anw0r.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@b4n requested changes on #2085"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/2085#pullrequestreview-201855022"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2085#pullrequestreview-201855022",
"url": "https://github.com/geany/geany/pull/2085#pullrequestreview-201855022",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>