New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update idris.tmLanguage with 0.99 syntax #11
Conversation
@@ -121,6 +121,12 @@ | |||
</array> | |||
</dict> | |||
<dict> | |||
<key>match</key> | |||
<string>(?<!\w)\'\w++(?!')</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What exactly the use case of this mandatory green match?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise it will match 'abcde
in text 'abcdef'
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you give me some examples of this syntax change? I just can not get my sense to embrace this change actually 😅.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Idris have introduced tagged effects using this syntax:
whatever : Eff ['a ::: STATE, 'b ::: STATE] ()
whatever = do
x <- ('a :- get)
'b :- (put x)
So we have to support 'identifier
syntax.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, just not catch up with the community for months, thanks @be5invis.
Hi @be5invis, sorry for the delay, I was busy these days and did not check out GitHub notifications recently 😅 And for the syntax file, I planned to sync this from atom-package, need to find a way to translate from CSON file to JSON file safely. Because that plugin will be catch up the changes of Idris itself and maintaining another tmLanguage file by our own seems redundant here. I will merge this first, and then we can discuss the proposal above later 😄 |
@zjhmale Hmm maybe I should add support for |
@zjhmale and |
Thanks for the tip! Actually, VSCode can also work with JSON-defined syntax as expected, so maybe just convert from CSON to JSON is kindly enough 😄 , what do you think?
|
No description provided.