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

incorrect-version Check Throws False Negative Error for 1.0-SNAPSHOT #237

Closed
volodya-lombrozo opened this issue Jan 14, 2025 · 10 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@volodya-lombrozo
Copy link
Member

XMIR might be generated from any source (not only from EO), by this reason sometimes we can generate XMIR by using jeo-maven-plugin:1.0-SNAPSHOT. Pay attention to the version 1.0-SNAPSHOT - it's correct semver version.

Semver BNF (part):

<valid semver> ::= <version core>
                 | <version core> "-" <pre-release>
                 | <version core> "+" <build>
                 | <version core> "-" <pre-release> "+" <build>

You can read more about it here: https://semver.org

But for the version 1.0-SNAPSHOT I'm getting the following error:

XMIR is incorrect: [[incorrect-version WARNING]:0 The format of the +version meta is wrong: "1.0-SNAPSHOT" (SemVer expected instead)], 

Please, remove this check or fix it.

@volodya-lombrozo
Copy link
Member Author

volodya-lombrozo commented Jan 14, 2025

@yegor256 Could you have a look, please? It blocks: objectionary/jeo-maven-plugin#962

@volodya-lombrozo
Copy link
Member Author

Well, with 0.1.0-SNAPSHOT version it works fine. So, actually 1.0-SNAPSHOT doesn't follow semver according with BNF I send before. My bad. I need to change the version in my project.

However, I have some doubts about it. 1.0-SNAPSHOT looks correct to me, although it doesn't follow semver.

@yegor256 What do you think if we allow such versions?

@yegor256
Copy link
Member

@volodya-lombrozo I agree, we should allow x.x-SNAPSHOT (it's Maven convention)

@yegor256
Copy link
Member

@h1alexbel can you help please?

@yegor256 yegor256 added the bug Something isn't working label Jan 14, 2025
@h1alexbel
Copy link
Contributor

@yegor256 yes, I'm in

h1alexbel added a commit to h1alexbel/lints that referenced this issue Jan 14, 2025
h1alexbel added a commit to h1alexbel/lints that referenced this issue Jan 14, 2025
yegor256 added a commit that referenced this issue Jan 14, 2025
bug(#237): incorrect-version allows maven convention
@yegor256
Copy link
Member

@rultor release, tag is 0.0.32

@rultor
Copy link
Contributor

rultor commented Jan 14, 2025

@rultor release, tag is 0.0.32

@yegor256 OK, I will release it now. Please check the progress here.

@rultor
Copy link
Contributor

rultor commented Jan 14, 2025

@rultor release, tag is 0.0.32

@yegor256 Done! FYI, the full log is here (took me 12min).

@h1alexbel
Copy link
Contributor

@volodya-lombrozo should be fixed, try again, please

@volodya-lombrozo
Copy link
Member Author

@h1alexbel Thank you for the fast fix! Works like a charm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants