Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Differences in MINOR and PATCH parts are not reported #141

Open
rimutaka opened this issue May 19, 2021 · 10 comments
Open

Differences in MINOR and PATCH parts are not reported #141

rimutaka opened this issue May 19, 2021 · 10 comments
Assignees

Comments

@rimutaka
Copy link

A new version alert is not displayed if the difference is in MINOR or PATCH numbers only. It seems that only MAJOR version differences result in displaying the latest version number.

E.g., urlencoding has the following versions available:

1.3.3(docs)
1.2.0
1.1.1
1.1.0
1.0.0
0.2.0
0.1.1

Using urlencoding = "1.0.0" through to urlencoding = "1.3.3" prints a green tick and no latest version alert.
image

Dropping to urlencoding = "0.2" displays X 1.3.3 next to it.
image

Is this a bug or how it should work?

I would expect anything below urlencoding = "1.3.3" to have the latest version alert printed next to it.

Desktop (please complete the following information):

  • OS: Ubuntu 20
  • Version v0.5.9
@phoekz
Copy link

phoekz commented Jul 29, 2021

I agree. At least it could be an option whether or not minor/patch differences trigger the latest version alert.

@koutheir
Copy link

This looks like a regression, because I remember this used to work correctly, even when minor versions change.

@koutheir
Copy link

This regression makes this extension useless half of the time.

@serayuzgur
Copy link
Owner

This is working as expected. As much as wi know cargo works like that and extension works as they described. Please take a look at #86.

@koutheir
Copy link

So this seems to be the behavior by design, even if it was not like this before. I understand, but it still makes this extension useless half of the time though, whereas it was very useful before.

@rimutaka
Copy link
Author

I wouldn't go as far as saying the extension is useless, but I think it would be more consistent if it warned about any new versions not compatible with the set pattern. E.g.

TOML CRATES WARNING
1 1.3.3
1.3 1.3.3
1.2 1.3.3 X
1.3.1 1.3.3 X

In other words, if there is a newer version that would not be automatically downloaded by cargo update I'd like to be warned about it.

@serayuzgur , do you think it's a breaking change and we'll need a configuration setting for it to tun on/off?
Would you consider a PR or prefer to do it yourself? I have no experience with VSCode extension, but happy to give it a try.

@koutheir
Copy link

I wouldn't go as far as saying the extension is useless,...

I said the extension is useless half of the time, because that is half of the time the reason I open Cargo.toml with this extension activated, that is, to figure out if I need to upgrade some code in order to use a newer version of a dependency which is not semver-compatible. Dependencies that are semver-compatible are not interesting to me because they will always be automatically upgraded via my frequent cargo updates.

Again, I understand that this behavior is by design. Still, this makes the extension way less useful to me. I didn't mean to offend anybody, I simply stated the result of this design decision on my own use case.

@serayuzgur
Copy link
Owner

think it's a breaking change and we'll need a configuration setting for it to tun on/off?
Would you consider a PR or prefer to do it yourself? I have no experience with VSCode extension, but happy to give it a tr

I will check and implement it if it is simple to

@serayuzgur serayuzgur self-assigned this Jan 3, 2022
@Christoph-AK
Copy link

Hi, any progress on this?

Maybe add a less aggressive warning on this, maybe ⚠️ instead of ❌?

@koutheir
Copy link

3 years later, it still looks like nobody cares about this issue...

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

No branches or pull requests

5 participants