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
Incorrect version comparison #66
Comments
We need to write a unit test that duplicates this scenario before fixing it, so we can be sure we have the case isolated. |
The problem: Another case where we would see this: upstream pushes an older tag to the git repo In these case, anitya will announce the new release and will say on the website that something odd was found since the new version seems older than the previous latest version. |
So, is any fix needed in anitya? Or... should we just fix the language generated by fedmsg_meta_fedora_infrastructure? |
We can go both ways (but I think I'd like to keep it in anitya, this feature is helpful when a project has bugfix releases for older version, think 1.2.x while 2.y is out), a third option could be to publish a message saying |
Yeah, either option sounds good to me. To restate, either:
(honestly, I prefer option 1 since it will have the least amount of impact on other systems that might already be subscribed to anitya's message stream) |
+1 on the first option for me as well :-) -------- Message d'origine -------- Yeah, either option sounds good to me. To restate, either: Modify fedmsg_meta_fedora_infrastructure to have a nicer message when old bugfix releases are discovered. — |
By odd I mean that the new version found appears to be older than the last version we had (ie: upstream released a version 2 after having released a version 3). This should help fixing #66
Noting that this only requires a change to fedmsg_meta now to use that new field to make the description more nice. |
fedora-infra/fedmsg_meta_fedora_infrastructure@380d8c4
I just saw this message go by the fedmsg bus, which seems incorrect.
The text was updated successfully, but these errors were encountered: