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

Missing SerialPortStream project in unit test projects #86

Closed
ToddMOsborne opened this issue Feb 26, 2019 · 4 comments
Closed

Missing SerialPortStream project in unit test projects #86

ToddMOsborne opened this issue Feb 26, 2019 · 4 comments
Labels

Comments

@ToddMOsborne
Copy link

When I was compiling your main solution file, the unit tests failed to build before a project reference was not added to the main SerialPortStream assembly. I updated this and all is well. I also created .net 4.7.1 versions of all projects and solutions. Let me know if you are interested in having them. Thanks, works great so far!

@jcurl
Copy link
Owner

jcurl commented Feb 26, 2019

Which solution file are you referring to? The SerialPortStream-net40.sln and SerialPortStream-net45.sln work for me, when opening with VS2015. The SerialPortStream-netstandard15 loads with VS2017 and doesn't reference test cases (I don't have the time to support .NET Core, nor Mac).

@jcurl
Copy link
Owner

jcurl commented Feb 26, 2019

Also, for 4.7.1, any functional changes, or just reference updates? In which case, if there are no functional changes, I prefer to support the lowest available standard, as they can then be used by any newer version of .NET allowing the greatest possible deployment.

@ToddMOsborne
Copy link
Author

ToddMOsborne commented Feb 26, 2019 via email

@jcurl
Copy link
Owner

jcurl commented Mar 24, 2019

I'm not able to reproduce the issue when I check out the files on my system. I don't know what is different from yours.

For now, I need to keep with .NET 4.0 as that is what I need, and there is nothing needed to require .NET 4.7.1 (with .NET 4.5 already provided, it is using whatever version of .NET you've got installed).

@jcurl jcurl closed this as completed Mar 24, 2019
@jcurl jcurl added the invalid label Mar 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants