Skip to content

Hide the content files that come from the Microsoft.CodeAnalysis.Workspaces.Common package in the web.ui project #19239

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

Merged

Conversation

AndyButland
Copy link
Contributor

Prerequisites

  • I have added steps to test this contribution in the description below

Relates to this internal discussion at HQ.

Description

#19202 resolved restore warnings on the Web.UI project, but due to the upgrade of Microsoft.CodeAnalysis.Workspaces.Common introduced some "folders" that pollute the solution view.

Adding this update to the .csproj file hides them.

Testing

Verify that you no longer see these content files in the solution view:

image

…spaces.Common package in the web.ui project.
@Copilot Copilot AI review requested due to automatic review settings May 6, 2025 09:03
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • src/Umbraco.Web.UI/Umbraco.Web.UI.csproj: Language not supported

@AndyButland AndyButland changed the title Hide the content files that come from the Microsoft.CodeAnalysis.Workspaces.Common package in the web.ui project. Hide the content files that come from the Microsoft.CodeAnalysis.Workspaces.Common package in the web.ui project May 6, 2025
@nikolajlauridsen nikolajlauridsen enabled auto-merge (squash) May 6, 2025 09:05
@nikolajlauridsen nikolajlauridsen merged commit 0d080bd into release/16.0 May 6, 2025
22 checks passed
@nikolajlauridsen nikolajlauridsen deleted the v16/task/hide-build-folders-in-web-ui-project branch May 6, 2025 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants