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

Updated PackageReference from 4.7.5 to 4.8.0. #115

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

softwareantics
Copy link
Contributor

Description

  • Updated PackageReference for GLWpfControl.csproj.
  • Updated PackageReference for Example.csproj.

Fixes #114

Dependencies

Type of change

  • Bug fix (non-breaking change which fixes an issue).

How Has This Been Tested?

I tested the example locally on my machine and everything still needs to be working. Furthermore, I created my own local package and tested it in my own personal project to make sure that I would no longer receive the warnings described in #114.

image

Test Configuration:

  • Operating System: Windows 10 Home
  • Hardware: Intel i7-6700HQ, 32GB RAM, GTX 950M
  • Toolchain: VS Community 2022 17.5.4

Possible Issues

I don't know if this will directly fix the issue once a new release is made. I think there is something that the maintainers of the NuGet package will have to address unless I can be pointed in the right direction. When uploading my own package of the control to NuGet instead of checking if the version was between 4.3.0 and 5.0.0 it ensured that it was greater than or equal to 4.8.0.

image

@NogginBops
Copy link
Member

To update these references you also need to update the version in the paket file:
https://github.com/opentk/GLWpfControl/blob/master/src/GLWpfControl/paket

Updating this version will fix the nuget package version range.

@softwareantics
Copy link
Contributor Author

My new laptop charger is on its way, express shipping. Once it comes around I'll make this change and hopefully we can update the package :)

@NogginBops NogginBops added this to the 4.3.0 milestone Oct 26, 2023
@NogginBops NogginBops added the 4.0 label Oct 26, 2023
@NogginBops NogginBops mentioned this pull request Oct 26, 2023
@NogginBops NogginBops linked an issue Oct 26, 2023 that may be closed by this pull request
@NogginBops
Copy link
Member

@softwareantics ping if you forgot this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update the PackageReference for OpenTK to 4.8.0. Extra window created
2 participants