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

Fix centrally-versioned package references #4674

Merged
merged 4 commits into from
Aug 22, 2024

Conversation

JesseCol
Copy link
Contributor

For the projects under the VSIX folder, the build stopped honoring the "Directory.Packages.props" file at the root of the repo. This causes us to pick up old versions of all these packages, and hilarity ensues.

I'm not sure at the moment what caused this change in behavior, but this fix looks safe as we continue to investigate.

@JesseCol JesseCol changed the base branch from main to release/1.6-stable August 22, 2024 22:03
@kythant
Copy link
Contributor

kythant commented Aug 22, 2024

Do you have one for the main branch coming?

@JesseCol
Copy link
Contributor Author

JesseCol commented Aug 22, 2024

Do you have one for the main branch coming?

Not yet but I'll do it soon. Tracking with bug 53411362.

@JesseCol JesseCol merged commit abe89b5 into release/1.6-stable Aug 22, 2024
27 checks passed
@JesseCol JesseCol deleted the user/jecollin/fix-nuget-versions branch August 22, 2024 23:49
JesseCol added a commit that referenced this pull request Aug 22, 2024
* Use CentralPackageVersions in VSIX projects

* Turns out we just need to make sure to import the Directory.Build.props at the root

* Revert "Use CentralPackageVersions in VSIX projects"

This reverts commit 3446e0c.

* Don't include the entire root Directory.build.props
JesseCol added a commit that referenced this pull request Aug 23, 2024
* Use CentralPackageVersions in VSIX projects

* Turns out we just need to make sure to import the Directory.Build.props at the root

* Revert "Use CentralPackageVersions in VSIX projects"

This reverts commit 3446e0c.

* Don't include the entire root Directory.build.props
JesseCol added a commit that referenced this pull request Aug 23, 2024
* Use CentralPackageVersions in VSIX projects

* Turns out we just need to make sure to import the Directory.Build.props at the root

* Revert "Use CentralPackageVersions in VSIX projects"

This reverts commit 3446e0c.

* Don't include the entire root Directory.build.props
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