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

Groovy syntax highlighting not automatically applied for files named Jenkinsfile #106812

Closed
joshbolduc opened this issue Sep 16, 2020 · 1 comment · Fixed by #106813
Closed

Groovy syntax highlighting not automatically applied for files named Jenkinsfile #106812

joshbolduc opened this issue Sep 16, 2020 · 1 comment · Fixed by #106813
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders languages-basic Basic language support issues verified Verification succeeded

Comments

@joshbolduc
Copy link
Contributor

Issue Type: Bug

Repro steps:

  1. Open a file named Jenkinsfile

Expect:
Groovy file mode is detected automatically (worked in 1.48)

Actual:
File is detected as plain text (as of 1.49)

It looks like this change removed "filenames": [ "Jenkinsfile" ] from the groovy extension, presumably inadvertently. By way of comparison, the docker extension has both a filenames entry for Dockerfile and a pattern:

"filenames": [ "Dockerfile", "Containerfile" ],
"filenamePatterns": [ "Dockerfile.*", "Containerfile.*" ],

So I'm inferring that the fix would be to restore the deleted line and also retain the new pattern, which would make everyone happy. (Or as happy as they can be writing Jenkinsfiles, anyway.)

VS Code version: Code - Insiders 1.50.0-insider (e13875b, 2020-09-15T05:35:57.362Z)
OS version: Windows_NT x64 10.0.18363

Extensions: none

@vscodebot
Copy link

vscodebot bot commented Sep 16, 2020

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

joshbolduc added a commit to joshbolduc/vscode that referenced this issue Sep 16, 2020
@aeschli aeschli self-assigned this Sep 16, 2020
@aeschli aeschli added this to the September 2020 milestone Sep 16, 2020
@aeschli aeschli added languages-basic Basic language support issues bug Issue identified by VS Code Team member as probable bug labels Sep 16, 2020
@alexr00 alexr00 added the verified Verification succeeded label Oct 1, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Oct 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders languages-basic Basic language support issues verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants
@aeschli @joshbolduc @alexr00 and others