<p><b>@ntrel</b> commented on this pull request.</p>

<p>Providing this works for enough TypeScript code, I think we should merge this.<br>
<a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=3161395" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/jhpratt">@jhpratt</a> It would be good to resolve my review points though.</p><hr>

<p>In <a href="https://github.com/geany/geany/pull/1450#discussion_r291659865">data/filedefs/filetypes.TypeScript.conf</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,54 @@
+# based on JavaScript file
+# For complete documentation of this file, please see Geany's main documentation
+[styling=C]
+
+[keywords]
+# all items must be in one line
+primary=break case catch class const continue debugger default delete do else enum export extends extend false finally for function get if import in Infinity instanceof let NaN new null return set static super switch this throw true try typeof undefined var let while with yield prototype async await declare aliased interfaced Alias Interface interface
+secondary=Array Boolean boolean Date Function Math Number number Object String string RegExp EvalError Error RangeError ReferenceError SyntaxError TypeError URIError constructor prototype decodeURI decodeURIComponent encodeURI encodeURIComponent eval isFinite isNaN parseFloat parseInt protected public private keyof void any never readonly as
</pre>
<p>Each of these keyword lists are not sorted properly. I'm not sure if Scintilla requires sorted lists for correct operation.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1450#discussion_r291661914">data/filedefs/filetypes.TypeScript.conf</a>:</p>
<pre style='color:#555'>> +comment_use_indent=true
+
+# context action command (please see Geany's main documentation for details)
+context_action_cmd=
+
+[indentation]
+#width=4
+# 0 is spaces, 1 is tabs, 2 is tab & spaces
+#type=1
+
+[build-menu]
+# %f will be replaced by the complete filename
+# %e will be replaced by the filename without extension
+# (use only one of it at one time)
+#FT_02_LB=_Lint
+#FT_02_CM=jshint "%f"
</pre>
<p>Are there no commands to compile/check a typescript file has valid syntax?</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/1450?email_source=notifications&email_token=AAIOWJ2SMPMF5EYGQMLZBEDPZKC43A5CNFSM4DFRV3LKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB253UVA#pullrequestreview-247183956">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ5ZBFR2BBDMGZSFX63PZKC43ANCNFSM4DFRV3LA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJ2OJN6YX3IP66OZVUDPZKC43A5CNFSM4DFRV3LKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB253UVA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/1450?email_source=notifications\u0026email_token=AAIOWJ2SMPMF5EYGQMLZBEDPZKC43A5CNFSM4DFRV3LKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB253UVA#pullrequestreview-247183956",
"url": "https://github.com/geany/geany/pull/1450?email_source=notifications\u0026email_token=AAIOWJ2SMPMF5EYGQMLZBEDPZKC43A5CNFSM4DFRV3LKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB253UVA#pullrequestreview-247183956",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>