Skip to content

Commit

Permalink
Merge pull request #384 from CodinGame/vscode-1.88
Browse files Browse the repository at this point in the history
Update to VSCode 1.88
  • Loading branch information
CGNonofr authored Apr 9, 2024
2 parents 7b84fb7 + 16c7201 commit 11f634c
Show file tree
Hide file tree
Showing 57 changed files with 1,173 additions and 1,057 deletions.
58 changes: 30 additions & 28 deletions demo/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions demo/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,14 @@
"devDependencies": {
"@codingame/esbuild-import-meta-url-plugin": "^1.0.2",
"@codingame/monaco-vscode-rollup-vsix-plugin": "file:../dist/rollup-vsix-plugin",
"@types/dockerode": "^3.3.26",
"@types/dockerode": "^3.3.28",
"@types/express": "^4.17.21",
"@types/node": "^20.11.4",
"@types/wicg-file-system-access": "^2023.10.5",
"@types/ws": "^8.5.10",
"ts-node": "^10.9.2",
"typescript": "~5.4.3",
"vite": "~5.2.6"
"typescript": "~5.4.4",
"vite": "~5.2.8"
},
"dependencies": {
"@codingame/monaco-vscode-accessibility-service-override": "file:../dist/service-override-accessibility",
Expand Down
19 changes: 5 additions & 14 deletions demo/src/features/testing.ts
Original file line number Diff line number Diff line change
Expand Up @@ -266,20 +266,11 @@ void getApi().then(async api => {
run.appendOutput(`Completed ${test.id}\r\n`)
}

run.coverageProvider = {
provideFileCoverage () {
const coverage: vscode.FileCoverage[] = []
for (const [uri, statements] of coveredLines) {
coverage.push(
api.FileCoverage.fromDetails(
api.Uri.parse(uri),
statements.filter((s): s is vscode.StatementCoverage => s != null)
)
)
}

return coverage
}
for (const [uri, statements] of coveredLines) {
run.addCoverage(api.FileCoverage.fromDetails(
api.Uri.parse(uri),
statements.filter((s): s is vscode.StatementCoverage => s != null)
))
}

run.end()
Expand Down
2 changes: 1 addition & 1 deletion docs/vscode_monaco_upgrade.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
- Wait for the new vscode version to be downloaded and built
- Run `npm run update-vscode-dependencies` to update the vscode dependencies we use to the same versions as VSCode
- Run `npm install` (might need to update/add the npm override for `xterm` in the `package.json` to fix the invalid peer deps)
- Fix errors, adapt code, build, include the `vscode.patch` into this commit
- Fix errors, adapt code, build, include the `vscode-patches` directory into this commit
- Do not hesitate to run the eslint autofix, it gets rid of the majority of your errors
- Implement missing services. This is easily observable seem when running the demo (see next point)
- Update demo
Expand Down
Loading

1 comment on commit 11f634c

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.