microsoft / compose-language-service

Language service for Docker Compose documents
https://www.npmjs.com/package/@microsoft/compose-language-service
MIT License
53 stars 5 forks source link

!reset / !override / … not supported by scheme #150

Open AnrDaemon opened 6 days ago

AnrDaemon commented 6 days ago

Type: Bug

The !reset / !override tags are not supported and highlighted as errors.

Extension version: 1.29.3 VS Code version: Code 1.95.0 (912bb683695358a54ae0c670461738984cbb5b95, 2024-10-28T20:16:24.561Z) OS version: Windows_NT x64 10.0.19045 Modes:

System Info |Item|Value| |---|---| |CPUs|Intel(R) Core(TM) i3-8145U CPU @ 2.10GHz (4 x 2304)| |GPU Status|2d_canvas: enabled
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled
webnn: disabled_off| |Load (avg)|undefined| |Memory (System)|7.86GB (0.42GB free)| |Process Argv|--folder-uri file:///d%3A/Profiles/anrdaemon/Documents/hk/invapi-php| |Screen Reader|no| |VM|0%|

Example compose file:

services:
  nginx:
    ports: !override
      - '127.0.0.1:8080:80'
    build:
      args:
        RELEASE_VERSION: dev
        ENV: dev
        CONF: devenv.conf
    volumes:
      - "./app:/opt/app/app"
      - "./build/nginx/config.js:/opt/app/app/panel/config.js"
  php-fpm:
    build:
      target: devenv
    links: !reset []
    volumes:
      - '.:/opt/app'
      - './build/php-fpm/config.php:/opt/app/config.php'
    extra_hosts:
      - "host.docker.internal:192.168.1.11"
  rabbitmq: !reset null

Image

bwateratmsft commented 4 days ago

Moving this to the compose language service repo. Thanks for the report!

Graloth commented 3 days ago

I would also very much request the compose spec used by vscode to be updated, since YAML tags have been a thing for quite some time now as part of the normal YAML spec, and the compose spec has added support for them for quite some time as well.

Hopefully it'll be a relatively simple thing to just update the version of the compose spec used.

AnrDaemon commented 3 days ago

I would also very much appreciate code formatting. Thank you.