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

Split extension into PreVS2022 and VS2022 projects #13

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jizc
Copy link
Contributor

@jizc jizc commented Sep 3, 2021

Summary

This splits the extension in two; PreV2022 and VS2022. The PreVS2022 project targets VS2017 and VS2019, as before.

Both projects have been modernized to use PackageReference. Both projects now use the Microsoft.VisualStudio.SDK NuGet package, instead of all the separate legacy extension libraries. The only real differences between the projects are the versions they use of the NuGet packages and the vsixmanifest.

Shared code was placed in the new LineSorter.Shared project. Other shared files was placed in a SharedFiles folder, and linked into each project.

Notes

  • I also modernized the LineSorter.Export project to increase maintainability for the future
  • Removed the dependency on Microsoft.VisualStudio.MPF.15.0 because that is only needed pre-VS2010 (see blog post)
  • Removed the BackgroundProjectLoad_string ProvideAutoLoad attribute, as it was marked as obsolete in VS2019, and the other auto load attributes should be enough
  • Bumped the version to 5.0.1

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.

1 participant