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.

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

Wrong YAML syntax highlighting in some corner cases for strings #55355

Closed
yajo opened this issue Jul 30, 2018 · 4 comments
Closed

Wrong YAML syntax highlighting in some corner cases for strings #55355

yajo opened this issue Jul 30, 2018 · 4 comments
Assignees
Labels
languages-basic Basic language support issues

Comments

@yajo
Copy link

yajo commented Jul 30, 2018

Issue Type: Bug

Comments inside the image explain the wrong syntax highlighting problems; see:
captura de pantalla de 2018-07-30 10-20-20

VS Code version: Code 1.25.1 (1dfc5e5, 2018-07-11T15:40:20.190Z)
OS version: Linux x64 4.17.6-200.fc28.x86_64

System Info
Item Value
CPUs AMD FX-7500 Radeon R7, 10 Compute Cores 4C+6G (4 x 1722)
GPU Status 2d_canvas: unavailable_software
flash_3d: unavailable_software
flash_stage3d: unavailable_software
flash_stage3d_baseline: unavailable_software
gpu_compositing: unavailable_software
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: disabled_software
rasterization: unavailable_software
video_decode: unavailable_software
video_encode: unavailable_software
vpx_decode: unavailable_software
webgl: unavailable_off
webgl2: unavailable_off
Load (avg) 2, 2, 2
Memory (System) 14.63GB (0.15GB free)
Process Argv /usr/share/code/code --unity-launch
Screen Reader no
VM 0%
Extensions (21)
Extension Author (truncated) Version
EditorConfig Edi 0.12.4
fish Ted 0.0.4
vscode-markdownlint Dav 0.19.0
vscode-eslint dba 1.4.12
githistory don 0.4.2
xml Dot 2.3.1
gc-excelviewer Gra 2.1.25
beautify Hoo 1.3.2
restructuredtext lex 65.0.0
fish-ide lun 0.3.1
rainbow-csv mec 0.4.2
dotenv mik 1.0.1
vscode-apache mrm 1.1.1
language-gettext mro 0.1.1
vscode-language-pack-es MS- 1.25.2
python ms- 2018.7.1
azure-account ms- 0.4.2
vscode-docker Pet 0.1.0
qub-xml-vscode qub 1.2.8
sass-indented rob 1.4.9
jinja who 0.0.8
@vscodebot vscodebot bot added the terminal Integrated terminal issues label Jul 30, 2018
@Tyriar Tyriar added languages-basic Basic language support issues and removed terminal Integrated terminal issues labels Jul 30, 2018
@Tyriar Tyriar assigned aeschli and unassigned Tyriar Jul 30, 2018
@aeschli
Copy link
Contributor

aeschli commented Jul 30, 2018

@yajo Would you mind pasting the code example here? Thanks a lot!

@texastoland
Copy link

texastoland commented Jul 31, 2018

Same here:

key: string
  "flows" | here
#         ^^^^^^ highlighted

Surely related to textmate/yaml.tmbundle#17.

@yajo
Copy link
Author

yajo commented Jul 31, 2018

# Highlight here is OK
- name: build docker-compose project {{ project.key }}
  # Highlight is misleading. All of this block is a string
  split spaces: |
    - i'm a string
    - [not an array]
    - {nor a dict}
  # same here
  join spaces: >
    - i'm a string
    - [not an array]
    - {nor a dict}
  when:
    # This is all a single string
    - not build_cache|bool or "ERROR: for odoo" in result.stderr
  command:
    # Highlighting in line 42 is misleading. All of this is a string too
    docker-compose build --pull
    {% if not build_cache|bool %} --no-cache {% endif %}

@aeschli
Copy link
Contributor

aeschli commented Sep 14, 2018

This issue was moved to textmate/yaml.tmbundle#29

@aeschli aeschli closed this as completed Sep 14, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
languages-basic Basic language support issues
Projects
None yet
Development

No branches or pull requests

4 participants