Skip to content
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

False remove trailing quotes by Array Processor. #129

Closed
huxuan opened this issue Apr 11, 2023 · 1 comment · Fixed by #132
Closed

False remove trailing quotes by Array Processor. #129

huxuan opened this issue Apr 11, 2023 · 1 comment · Fixed by #132
Labels
bug Something isn't working

Comments

@huxuan
Copy link

huxuan commented Apr 11, 2023 •
edited
Loading

Example yaml:

section:
  - '*[a]'

When the Array Processor is enabled, the trailing single quote will be removed.
@huxuan huxuan changed the title False remove trailing quotes by ArrayProcessor. False remove trailing quotes by Array Processor. Apr 11, 2023
@pascalre pascalre added the bug Something isn't working label Apr 30, 2023
pascalre added a commit that referenced this issue Apr 30, 2023
@pascalre
Fix quoting issues of ArrayProcessor (#129)
b89e97b
@pascalre pascalre linked a pull request Apr 30, 2023 that will close this issue
Fix quoting issues of ArrayProcessor (#129) #132
Merged
pascalre added a commit that referenced this issue Apr 30, 2023
@pascalre
Merge pull request #132 from pascalre/129-false-remove-trailing-quote…
ec88890 
…s-by-array-processor

Fix quoting issues of ArrayProcessor (#129)
@pascalre
Copy link
Owner

pascalre commented May 1, 2023

@huxuan check out 6.5.2, which fixes this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix quoting issues of ArrayProcessor (#129) pascalre/vscode-yaml-sort
2 participants
@huxuan @pascalre