Skip to content
This repository was archived by the owner on Oct 2, 2022. It is now read-only.

wrong engine being used #78

Closed
wants to merge 1 commit into from
Closed

wrong engine being used #78

wants to merge 1 commit into from

Conversation

maapteh
Copy link

@maapteh maapteh commented Jul 27, 2020

fixes #77

error @endemolshinegroup/[email protected]: The engine "yarn" is incompatible with this module. Expected version ">=1.3.0".
error Found incompatible module

@maapteh maapteh requested a review from a team as a code owner July 27, 2020 08:53
@maapteh maapteh changed the title Update package.json wrong engine being used Jul 27, 2020
@hassankhan
Copy link
Collaborator

Fixed in 09983de

@hassankhan hassankhan closed this Sep 5, 2020
@maapteh
Copy link
Author

maapteh commented Sep 5, 2020

@maapteh maapteh deleted the patch-1 branch September 5, 2020 07:50
@maapteh
Copy link
Author

maapteh commented Sep 5, 2020

@hassankhan pls check your changes, they are wrong

@hassankhan
Copy link
Collaborator

hassankhan commented Sep 5, 2020

Hi @maapteh,

Thanks for the update. In #77, you said the issue was that v1.3.0 does not exist. I looked through the available versions from npmjs.com and picked the closest version that did exist.

Do you by any chance have a test repo I can use to see the issue for myself?

@maapteh
Copy link
Author

maapteh commented Sep 5, 2020

I also mentioned that maybe you meant NPM and not YARN

the issue happens when you use your PUBLISHED npm library with yarn then it reads its package json and has the engine set to a yarn version which is not existing by facebook at all :) the engine is used when installing with yarn for its minimal requirements. So you want node version at least 10. Thats fine. But look what you write at yarn, thats not npm its yarn and yarn is now at 1.22 and your module for sure dont need this latest. It will work with any :)

@maapteh
Copy link
Author

maapteh commented Sep 5, 2020

Just remove that line, what i did in my PR which you then not took over. Good luck!

@hassankhan
Copy link
Collaborator

Ahhh right I see now, thanks for the explanation, I hadn't even realized that it could be an issue in that way 😅 will fix up shortly 👍

@hassankhan
Copy link
Collaborator

Fixed in 26e9aec

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

Successfully merging this pull request may close these issues.

wrong engine for yarn set :(
2 participants