-
Notifications
You must be signed in to change notification settings - Fork 889
comment-format: whitelist //region comments #4609
Conversation
Thanks for your interest in palantir/tslint, @01e9! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request. |
Hi @01e9, can you please provide more information around this code change? There is no associated issue and I don't know much about JetBrains IDE. I need to know a lot more context before I can consider this PR. |
@adidahiya JetBrains IDEs have this feature in menu "Code > Surround with > region" https://streamable.com/4sxnz |
Which versions of JetBrains IDE does this apply to? You'll have to update the code comment on the previous line too. |
I tried to find some JetBrains docs about this comment but without success. |
https://blog.jetbrains.com/phpstorm/2012/03/new-in-4-0-custom-code-folding-regions/ As you can see there are two custom folding items: “<editor-fold..>” for NetBeans-like style and “region..endregion” for VisualStudio style. |
//region
without space
How soon will this be included in release? Now I fix manually the |
PR checklist
Overview of change:
Continuation of #197
JetBrains IDE screencast https://streamable.com/4sxnz
CHANGELOG.md entry:
[enhancement]
comment-format
rule now whitelists//region
comments generated by JetBrains IDE