Merge pull request #5244 from dotty-staging/fix/5242

nicolasstucki · web-flow · commit 863d9442e75f · 2018-10-11T16:46:55.000+02:00
Fix #5242: Don't run all Scala files as worksheets
diff --git a/vscode-dotty/src/features.ts b/vscode-dotty/src/features.ts
@@ -48,7 +48,7 @@ export class WorksheetRunFeature extends TextDocumentFeature<TextDocumentRegistr
       return
     }
 
-    const selector: DocumentSelector = documentSelector || [ { language: 'scala', pattern: '**/*.sc' } ]
+    const selector: DocumentSelector = [ { language: 'scala', pattern: '**/*.sc' } ]
     this.register(this.messages, {
       id: generateUuid(),
       registerOptions: { documentSelector: selector }

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ export class WorksheetRunFeature extends TextDocumentFeature<TextDocumentRegistr`
`48`	`48`	`return`
`49`	`49`	`}`
`50`	`50`
`51`		`- const selector: DocumentSelector = documentSelector \|\| [ { language: 'scala', pattern: '*/.sc' } ]`
	`51`	`+ const selector: DocumentSelector = [ { language: 'scala', pattern: '*/.sc' } ]`
`52`	`52`	`this.register(this.messages, {`
`53`	`53`	`id: generateUuid(),`
`54`	`54`	`registerOptions: { documentSelector: selector }`