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

Extension IDs listed for 'unwantedRecommendations' should be ignored. #17

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

rgrunber
Copy link
Member

@rgrunber rgrunber changed the title Support 'unwantedRecommendations' in .vscode/extensions.json. Extension IDs listed for 'unwantedRecommendations' should be ignored. Sep 11, 2024
@rgrunber
Copy link
Member Author

@robstryker I think it should be good, but let me know if you see anything odd.

I'll probably have this merged, and take on the long journey of updating all consumers (but probably vscode-java first as the request originates from there).

@rgrunber rgrunber force-pushed the unwantedRecommendations branch 2 times, most recently from b9fc85d to 6b9abe7 Compare September 16, 2024 15:43
@robstryker
Copy link
Collaborator

https://github.com/redhat-developer/vscode-extension-proposals/pull/17/files#diff-edb612d4f1a6f7395a50700ab8b272110e81b8a1d92285aca5b23e54ed62c136R160

This looks like it will read the workspace file n times. Surely this can be optimized?

- Support 'unwantedRecommendations' in .vscode/extensions.json and in
  the '.code-workspace' (multi-root) files

Signed-off-by: Roland Grunberg <[email protected]>
@rgrunber
Copy link
Member Author

Added an optional argument for the json data that's read from the configuration file so it can be re-used.

.map((x) => x.extensionId)
.filter(filterUnique)
.filter((x) => !isExtensionInstalled(x));

const jsonData = await readExtensionConfigurationFile();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Much Improved.

@rgrunber rgrunber merged commit 20b4e01 into redhat-developer:main Sep 17, 2024
1 check failed
@rgrunber rgrunber deleted the unwantedRecommendations branch September 17, 2024 15:26
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.

Honor unwantedRecommendations setting in .vscode/extensions.json.
2 participants