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

Mark IsExternalInit as private assets #9153

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Mark IsExternalInit as private assets #9153

merged 1 commit into from
Jul 11, 2023

Conversation

smitpatel
Copy link
Member

@smitpatel smitpatel commented Jul 11, 2023

Moving to CPVM requires that extra properties specified on actual package reference now

This package ended up in our package and is blocking our VS insertion because we are using newer version.

Microsoft Reviewers: Open in CodeFlow

Moving to CPVM requires that extra properties specified on actual package reference now
@smitpatel smitpatel requested a review from a team as a code owner July 11, 2023 19:28
@drewnoakes
Copy link
Member

This package ended up in our package and is blocking our VS insertion because we are using newer version.

Which package? We have units tests that validate the files in our package. Maybe those tests aren't working, or something else was going on.

@smitpatel
Copy link
Member Author

"IsExternalInit" package 😄

@smitpatel smitpatel merged commit 54fbc0d into main Jul 11, 2023
6 checks passed
@smitpatel smitpatel deleted the smit/fixVSInsertion branch July 11, 2023 23:37
@ghost ghost added this to the 17.7 milestone Jul 11, 2023
@smitpatel
Copy link
Member Author

Would you expect the missing private assets to be caught by tests? I can look into tests to see what is going on.

@drewnoakes
Copy link
Member

This package ended up in our package

"IsExternalInit" package 😄

Sorry I meant which VSIX package (the second kind of package you mentioned). If the file was in the VSIX (which is just a ZIP file) then I would expect any change there (either adding it or removing it) to break a unit test.

Check PackageContentTests and the *.verified.txt files here:

https://github.com/dotnet/project-system/tree/main/tests/Microsoft.VisualStudio.ProjectSystem.Managed.VS.UnitTests/Setup

@drewnoakes
Copy link
Member

Ah, I see that the IsExternalInit pacakge is a source-only package.

https://github.com/manuelroemer/IsExternalInit/blob/master/src/IsExternalInit.nuspec

This means that someone else is also incorrectly declaring it as a (non-private) dependency.

@drewnoakes drewnoakes modified the milestones: 17.7, 17.8 Jul 13, 2023
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