Skip to content

enable producing reference assemblies by default#8111

Merged
rainersigwald merged 1 commit intodotnet:mainfrom
YuliiaKovalova:main
Nov 2, 2022
Merged

enable producing reference assemblies by default#8111
rainersigwald merged 1 commit intodotnet:mainfrom
YuliiaKovalova:main

Conversation

@YuliiaKovalova
Copy link
Copy Markdown
Member

This change allows producing reference assemblies by default for both legacy and SDK project systems

The corresponding changes were merged for legacy project system
https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1341885

This change allows producing reference assemblies by default for both legacy and SDK project systems
@drewnoakes
Copy link
Copy Markdown
Member

This is very exciting!

We need to ensure this change will only apply in VS when your corresponding CSPROJ change is active.

@rainersigwald
Copy link
Copy Markdown
Member

We need to ensure this change will only apply in VS when your corresponding CSPROJ change is active.

Since this file should only be shipped to VS in changes post-csproj-merge, I think it's ok? There is a slight possibility of someone explicitly using a new common.targets with an older VS (mostly internally in CoreXT environments) but that generally doesn't work anyway, so I think this is ok.

@rokonec rokonec added the merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now. label Nov 2, 2022
@rainersigwald rainersigwald merged commit 8016484 into dotnet:main Nov 2, 2022
rokonec added a commit that referenced this pull request Nov 4, 2022
rokonec added a commit that referenced this pull request Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants