Skip to content

Update to VScode 1.101.0 #639

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

onora-hbl
Copy link
Collaborator

I hope what I've done is correct 😅

I tested all the mode combinations in the demo, doing all the actions I had in mind. I didn't see an error anywhere

@onora-hbl onora-hbl requested a review from CGNonofr June 18, 2025 15:48
@onora-hbl onora-hbl self-assigned this Jun 18, 2025
Copy link
Contributor

@CGNonofr CGNonofr left a comment

Choose a reason for hiding this comment

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

Well done! looks really good

'vs/workbench/services/extensionManagement/common/media/defaultIcon.png': new URL(
'../../vscode/src/vs/workbench/services/extensionManagement/common/media/defaultIcon.png',
import.meta.url
).toString(),
Copy link
Contributor

Choose a reason for hiding this comment

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

What happened to that icon? 🤔

Copy link
Collaborator Author

@onora-hbl onora-hbl Jun 20, 2025

Choose a reason for hiding this comment

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

It was removed in this commit

@@ -76,7 +76,6 @@ import getSurveyServiceOverride from '@codingame/monaco-vscode-survey-service-ov
import getUpdateServiceOverride from '@codingame/monaco-vscode-update-service-override'
import getExplorerServiceOverride from '@codingame/monaco-vscode-explorer-service-override'
import getLocalizationServiceOverride from '@codingame/monaco-vscode-localization-service-override'
import getTreeSitterServiceOverride from '@codingame/monaco-vscode-treesitter-service-override'
Copy link
Contributor

Choose a reason for hiding this comment

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

What happened to that service override?

I'm still able to find related services in the VSCode code base, like ITreeSitterLibraryService or ITreeSitterThemeService

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I had missed these 2 new services they added. Here's the fix (which rollback the removal of the service override): 1cdba72

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants