Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Convert the C# sample projects to PackageReference format #533

Merged
merged 2 commits into from
Feb 6, 2021

Conversation

Numpsy
Copy link
Contributor

@Numpsy Numpsy commented Oct 27, 2020

Part one of the suggestion from #526 - convert the C# sample projects from packages.config to PackageReference format (and delete all the packages.config files)

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

@piksel piksel self-assigned this Nov 1, 2020
@piksel piksel self-requested a review November 1, 2020 17:56
Copy link
Member

@piksel piksel left a comment

Choose a reason for hiding this comment

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

LGTM. Added suggestions for updating to the just released v1.3.1

@Numpsy
Copy link
Contributor Author

Numpsy commented Dec 5, 2020

It should also be possible to put the SharpZipLib package version and/or the PackageReference in a shared .props file (such that the version can be updated in once place rather than having to change every project file). Not sure if that's a good idea or not though.

@piksel piksel merged commit 33da791 into icsharpcode:master Feb 6, 2021
@Numpsy Numpsy deleted the rw/samples/package_reference branch February 7, 2021 19:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants