Vscode-docker: WORKDIR paths should be absolute

[doggy8088]

1. F1 > Docker: Add Docker Files to Workspace
2. ASP․NET Core

The generated Dockerfile has an error on WORKDIR "/src/.". See below:

I have to remove double quote manually to remove the warning.

Is this kind of bug in vscode-docker?

[rcjsuen]

@doggy8088 This is an issue with the linter where it isn't taking the quotation marks into account correctly. I have opened rcjsuen/dockerfile-utils#67 to track this.

Thank you for your bug report and have a great day!

[doggy8088]

@rcjsuen Thanks!

[bwateratmsft]

@rcjsuen Would we need a new version of dockerfile-utils published to consume the fix in https://github.com/rcjsuen/dockerfile-utils/commit/11ef5d225d05bc79cb6435a78c89123e8efc03e1?

And/or dockerfile-language-server-nodejs?

[rcjsuen]

@bwateratmsft You'll need a new version of the language server to get the fix. When are you planning to cut a new release?

[bwateratmsft]

The powers that be tell me sometime in the next couple weeks. @BigMorty to keep me honest.

[BigMorty]

We are actually planning a small release fairly soon, how soon could we integrate a new version of the language server with this fix? Otherwise the next release will be a number weeks out; we don't have a specific date at this time.

[rcjsuen]

@BigMorty The size of the pull request is often rather miniscule (as the change is really a simple change to the package.json file). Let me tie up some loose ends and then I'll push out a new release for you.

[rcjsuen]

@bwateratmsft @BigMorty Opened #1619 to upgrade the language server.

[bwateratmsft]

Should be fixed now by https://github.com/microsoft/vscode-docker/pull/1619.

[bwateratmsft]

Verified this is fixed.

[BigMorty]

@bwateratmsft, I assume we will integrate the new language server in our next major release?

[bwateratmsft]

@BigMorty Yep!

[bwateratmsft]

The fix for this is now available in Docker extension version 1.0.0.