Skip to content

Fixed Dockerfile syntax highlighting#8991

Merged
TravisEz13 merged 5 commits intoPowerShell:masterfrom
RDIL:patch-3
Mar 11, 2019
Merged

Fixed Dockerfile syntax highlighting#8991
TravisEz13 merged 5 commits intoPowerShell:masterfrom
RDIL:patch-3

Conversation

@RDIL
Copy link
Contributor

@RDIL RDIL commented Feb 26, 2019

PR Summary

The f in Dockerfile is not uppercase, this breaks syntax highlighting for vscode on my client

PR Context

DockerFile breaks syntax highlighting as mentioned and described above.

PR Checklist

@TravisEz13
Copy link
Member

This affects our release build. I'll need to clone the branch into our release build system and verify this does not cause an issue. It will take some time to validate. Please be patient

@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 28, 2019
@RDIL
Copy link
Contributor Author

RDIL commented Feb 28, 2019

Not a problem.

@TravisEz13 TravisEz13 merged commit d63ac89 into PowerShell:master Mar 11, 2019
@RDIL RDIL deleted the patch-3 branch March 11, 2019 22:25
@adityapatwardhan adityapatwardhan added this to the 6.2.0 milestone Mar 13, 2019
TravisEz13 pushed a commit that referenced this pull request Mar 13, 2019
The `f` in Dockerfile is not uppercase, this breaks syntax highlighting for vscode on my client
<!-- Summarize your PR between here and the checklist. -->

## PR Context
`DockerFile` breaks syntax highlighting as mentioned and described above.
<!-- Provide a little reasoning as to why this Pull Request helps and why you have opened it. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants