Renovate can't update pnpm-lock.yaml when there's a peer dependencies error #15410
Answered
by
rarkins
philbarresi
asked this question in
Request Help
-
How are you running Renovate?WhiteSource Renovate hosted app on github.com If you're self-hosting Renovate, tell us what version of Renovate you run.No response Please select which platform you are using if self-hosting.github.com If you're self-hosting Renovate, tell us what version of the platform you run.No response Was this something which used to work for you, and then stopped?I never saw this working Describe the bugIf A warning due to minor version of Relevant debug logsLogsHave you created a minimal reproduction repository?No reproduction repository |
Beta Was this translation helpful? Give feedback.
Answered by
rarkins
May 3, 2022
Replies: 7 comments 14 replies
-
|
please provide some more context debug log lines when the error occured. |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Hi there, We have found that there's a problem with the logs. Depending on which situation applies follow one, some or all of these instructions. No logs at allIf there's no log posted yet, we need you to find and copy/paste the log into the issue template. Finding logs on hosted appClick me to read instructionsIf you use the Renovate app (GitHub):
Finding logs when self-hostingClick me to read instructionsIf you're running self-hosted, run with Insufficient logsClick me to read instructionsIf you already provided logs, and the Renovate team said they are not enough, follow the instructions from the No logs at all section. Formatting your logsClick me to read instructionsPlease put your logs in a |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Are you able to get to logs linked directly? This is a sample execution that failed: https://app.renovatebot.com/dashboard#github/thinkalpha/platform-ui/653089473 |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Nevermind, I see that you have a preferred way of including logs. Here's some context from the logs from that run, since I can't paste the full log in: Click me to see logs |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Hi @bytenik. Your analysis was mistaken, so I have deleted the invalid bug report, converted to a discussion, and renamed it. If you look into the stdout/stderr you pasted you can see the real cause: Extract: The outdated npm used to install |
Beta Was this translation helpful? Give feedback.
14 replies
-
|
Soon: |
Beta Was this translation helpful? Give feedback.
-
|
We already are using the package.json property It comes with the added benefit of being supported by Could this be supported? |
Beta Was this translation helpful? Give feedback.
-
|
It should already be: renovate/lib/modules/manager/npm/post-update/pnpm.ts Lines 106 to 113 in e97974d If you have specified |
Beta Was this translation helpful? Give feedback.
-
How to set |
Beta Was this translation helpful? Give feedback.
-
|
It's not a renovate setting |
Beta Was this translation helpful? Give feedback.
Answer selected by
rarkins
-
|
@nightah has confirmed this was a change in |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
If you want to stay on pnpm v6 and take some time to upgrade you could add this to your "constraints": {
"pnpm": "6"
},I just did that and from debug logs it resolves correctly to latest v6 after, which fixes the problem until we upgrade to v7 and remove this. See https://docs.renovatebot.com/configuration-options/#constraints |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @bytenik. Your analysis was mistaken, so I have deleted the invalid bug report, converted to a discussion, and renamed it.
If you look into the stdout/stderr you pasted you can see the real cause:
ERR_PNPM_PEER_DEP_ISSUES Unmet peer dependencies:Extract:
The outdated npm used to install
pnpmis not relevant.