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.

Sign up for GitHub

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

Jump to bottom

Update to older version #3720

Closed
josesimoes opened this issue May 14, 2021 · 3 comments
Closed

Update to older version #3720

josesimoes opened this issue May 14, 2021 · 3 comments
Labels
T: bug 🐞 Something isn't working

Comments

@josesimoes
Copy link

Package ecosystem
NuGet
Package manager version
NuGet
Language version
.NET C#
Manifest location and content prior to update

https://github.com/nanoframework/nanoFramework.TestFramework/blob/develop/.github/dependabot.yml

Updated dependency

Package is: nanoFramework.Tools.Debugger.Net
Current version was "2.0.0-preview.15" and dependabot updated it to "2.0.0-preview.8".

Native package manager behavior

See here: nanoframework/nanoFramework.TestFramework#48
@josesimoes josesimoes added the T: bug 🐞 Something isn't working label May 14, 2021
@asciimike
Copy link
Contributor

Weird... I wonder if we're doing a string compare as opposed to a number compare here... 😅

@mynkow
Copy link

mynkow commented May 16, 2021

Yes, dependabot is doing SemVer1.0 comparison. This issue has been already identified in the following (and probably there are many more) tickets:

#1972
#3643

@asciimike
Copy link
Contributor

Closing in favor of #1972, I'll see if someone can poke at this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T: bug 🐞 Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mynkow @josesimoes @asciimike