Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@mus65
Copy link
Contributor

@mus65 mus65 commented Sep 8, 2024

This migrates to Central Package Management . Versions stay the same, so there is no functional difference.

Additional changes I made as part of this:

  • replaced some usages of the [1.0.0,) syntax with 1.0.0 since these are equivalent, see docs
  • removed the testcontainers -> SSH.NET exclude from IntegrationTests . I can not reproduce this issue anymore. Even if it still woud be a problem, I think it would be better to use VersionOverride here since the version has no meaning anyway.

@Rob-Hague
Copy link
Collaborator

Just want to double check the exclude, not sure why there would be no build warnings anymore

@mus65
Copy link
Contributor Author

mus65 commented Sep 11, 2024

I can only reproduce the original warning with Testcontainers <= 3.5.0 which depended on SSH.NET 2020.0.2 . With Testcontainers >= 3.6.0 (which depends on SSH.NET 2023.0.0), the warning disappears.

Still no idea why this makes a difference though.

Directory.Packages.props Show resolved Hide resolved
Directory.Packages.props Show resolved Hide resolved
Copy link
Collaborator

@Rob-Hague Rob-Hague left a comment

Choose a reason for hiding this comment

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

Thanks for checking the build warnings. Let's be optimistic that it won't reoccur

@Rob-Hague Rob-Hague merged commit dde5f67 into sshnet:develop Sep 11, 2024
@mus65 mus65 deleted the centralpackagemanagement branch September 14, 2024 09:30
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.