-
Notifications
You must be signed in to change notification settings - Fork 81
Add color to more keywords #299
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
Conversation
Look good to me. If no reaction, I will merge it tomorrow |
Currently testing locally, but this seems a clear improvement worth merging :-) |
FWIW, highlighting "built-in types" has effects like and there was already some concern in #298 (comment). But on balance, this remains a net improvement that we can tweak later... |
@Blaisorblade can I merge? |
@dlesbre you are still doing some fixes I, great! Please tell me when you think it is ok and I will merge. |
I've been adding missing stuff as I noticed it missing since I opened the PR. The last update were just somethings I noticed today. I think you can merge this now if you want too. I'll open a new PR if I find enough missing things to warrant that. |
@dlesbre Perfect. Thanks a lot! |
This fixes #298
Adds missing syntax colors to:
Instance
,Program Instance
,Coercion
,Context
,Include
,Search
,Structure
,Generalizable All
,Generalizable Variables
Existing Instance
,Existing Class
,Variant
,About
,Locate
,Inline
(if afterVariable
,Hypothese
,...),Let Fixpoint
,Let CoFixpoint
,Canonical
, theObligation
commands, someProof
commands (Save
,Abort
,Undo
,Restart
,Focus
...),Implicit Types
,Typeclasses Opaque
,Typeclasses Transparent
fix ... for
andwhere ... and
.{|
and|}
nat
,bool
,option
,list
,unit
,sum
,prod
,comparison
,Empty_set
tt
,true
,false
,Some
,None
,S
,O
,inl
,inr
,pair
,cons
,Eq
,Lt
,Gt
,id
,ex
,all
andunique
.transitivity
,etransitivity
,f_equal
,exfalso
,replace
,tauto
,cycle
,swap
,revgoals
,cycle
,shelve
,unshelve
do
(move from tactic to keyword),tryif
once
,at
,only
Infix
,match
,Class
/Record
,Instance
,Existing Instance
,Generalizable ...
,pose
,destruct
andreplace
Variable (x: type).
,Context {a: type}
andContext `{a:type}
not coloring.Inductive > foo
not coloring.Class
andRecord
similarly toInductive
This includes all changes from #279.