[BUG] Version passed to cli ignored

[mathieutu]

Hi, I'm on a php project, and want to give the version manually.

$ gitmoji-changelog 1.2.0                                                                                                                                      Projects/laravel-json-syncer master
▶  start     gitmoji-changelog v2.1.0
ℹ  info      update ./CHANGELOG.md
ℹ  info      No custom configuration found
ℹ  info      use preset node
✖  error     Error: Cannot retrieve the version from your configuration. Check it or you can do "gitmoji-changelog <wanted version>". 
    at main (/Users/mathieutu/Projects/laravel-json-syncer/node_modules/gitmoji-changelog/src/cli.js:63:13)

you can do "gitmoji-changelog "

Actually this is exactly what I'm doing.

It's working on v1.

Thanks.

[frinyvonnick]

Hi @mathieutu 👋

Thank you for reporting this issue. Can I reproduce it on mathieutu/php-project-launcher?

[mathieutu]

Hi @frinyvonnick,
Yeah totally! As you will see, I've forced the version 1 in the package.json.

Thanks for your help.

[frinyvonnick]

I'll check this as soon as possible. Thank you for your answer 👍

[chengcyber]

It should be fixed by this PR #192

Workaround

In your project, manually freeze the version of read-pkg-up below 6.0.0, such as

npm install --save-dev --exact read-pkg-up@5.0.0

[mathieutu]

Actually this seems to be a totally different issue, and will be fixed after #190 merged.

[frinyvonnick]

This is available in canary version (gitmoji-changelog@2.2.0-alpha.24332f02). @chengcyber can you give it a try and tell me if it fixes your issue? Otherwise I'll check your PR but I don't understand why the dependency version is an issue since I didn't updated this dependency 🤔

[chengcyber]

@frinyvonnick That might be a package version resolution issue. In my project, jest is installed first, and it depends on read-pkg-up@7.0.1, since it installed before gitmoji-changelog. so the read-pkg-up in top-level is read-pkg-up@7.0.1

the dep tree kind of like this:

├─ @gitmoji-changelog/core@2.1.0
│  └─ read-pkg-up@4.0.0
├─ jest-resolve@26.1.0
│  └─ read-pkg-up@7.0.1
└─ read-pkg-up@7.0.1

[andybirdgir]

I fixed it！thank you so much Le ven. 9 oct. 2020 17:48, Cheng <notifications@github.com> a écrit :

…

@frinyvonnick <https://github.com/frinyvonnick> That might be a package version resolution issue. In my project, jest is installed first, and it depends on ***@***.***, since it installed before gitmoji-changelog. so the read-pkg-up in top-level is ***@***.*** the dep tree kind of like this: ├─ @***@***.*** │ └─ ***@***.*** └─ ***@***.*** — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#181 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AMI5PZ7U42OEYB7NUFYQYULSJ3L6DANCNFSM4K5PFJHQ> .