Skip to content

incorrect bracketing "(//))" #148

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.

Sign up for GitHub

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

Jump to bottom
Closed
duhd1993 opened this issue Sep 23, 2019 · 4 comments · Fixed by #167
Closed

incorrect bracketing "(//))" #148

duhd1993 opened this issue Sep 23, 2019 · 4 comments · Fixed by #167
Labels
bug status:fixed-waiting-merge

Comments

@duhd1993
Copy link

An additional round bracket is inserted. (//) this syntax is used for assigning values for an array.
rec
Is this controlled by this extension?
@pedro-ricardo
Copy link
Collaborator

Yes it is ! Thank you for the feedback.

This is the part that controls it... It should be simple but it seems to be accumulating rules.
Screenshot_2019-09-27_19-30-02

I'll have a look into it.

@pedro-ricardo
Copy link
Collaborator

This change in language_configuration.json appears to fix it ...
Screenshot_2019-09-27_19-36-04

Due to the first bracket is automatic closed, the close option of the array constructor don't need to have the ) again. But i can't prove that it will work all the time yet

@pedro-ricardo
Copy link
Collaborator

By the way ... The usage of (\ \) as array constructor is outdated, I suggest you to use the new symbol: [ ].

@duhd1993
Copy link
Author

Ok. Thank you for the update and suggestion.
This is not perfect, but looks like this is what you can do here.

@pedro-ricardo pedro-ricardo mentioned this issue Feb 24, 2020
Closed
12 tasks
@pedro-ricardo pedro-ricardo mentioned this issue Feb 27, 2020
Merged
13 tasks
pedro-ricardo added a commit that referenced this issue May 15, 2020
@pedro-ricardo
Minor release - version 2.2.0
db74870 
### Check List
- Highlight
- [x] `select` keyword - *Merge #165 to fix #149*
- [x] `rank` keyword -  *Merge #165*
- [x] `forall` keyword - *Merge #170 to fix #169*
- Symbol List
- [x] Symbols bug in VSCode +1.40 - *Merge #161 to fix #154*
- [x] Symbols classification - *Merge #141 to fix #140*
- [x] Long named symbols - *Merge #151*
- Settings
- [x] Add more options - *Merge #152*
- Dependencies
- [x] Merge `dependabot` PR's
    - [x] lodash - *Merge #144*
    - [x] mixin-deep - *Merge #146*
    - [x] handlebars - *Merge #155* 
- Code Completion
- [x] Array constructor - *Fix #148*
- New Features
- [x] Breakpoints - *Merge #150*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug status:fixed-waiting-merge
Projects
None yet
Milestone
No milestone
2 participants
@duhd1993 @pedro-ricardo