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

Lerna root "package-lock.json" not updated during child package renovation. #3317

Closed
abernix opened this issue Mar 7, 2019 · 6 comments
Closed
Labels
auto:reproduction A minimal reproduction is necessary to proceed help wanted Help is needed or welcomed on this issue manager:npm package.json files (npm/yarn/pnpm) stale status:requirements Full requirements are not yet known, so implementation should not be started type:bug Bug fix of existing functionality

Comments

@abernix
Copy link
Contributor

abernix commented Mar 7, 2019

What Renovate type are you using?
Renovate GitHub App

Describe the bug
I'm going to try to make a reproduction for this, but I believe I've seen an issue in a Lerna 3 monorepo which uses file: reference specifiers and does not have package-lock.json files in the nested packages. For example, the Lerna monorepo looks roughly like:

- package.json
- package-lock.json
- packages/
  - packageA/
    - package.json

In the case that I'm analyzing right now — which happened in a private repository — Renovate updated a package listed in the dependencies of packageA's package.json. Again, that packageA does not have a package-lock.json. In the past, this type of renovation has resulted in the updating of the root's package-lock.json, as is the case with this public issue which roughly represents the same type of update: apollographql/apollo-server#2004.

On this recent change though, the only changes to the root's package-lock.json were the toggling of the "optional": true attributes (which I know we're all too familiar with).

This last bit indicates to me that package-lock.json was updated, but just the update to the appropriate package didn't get caught, for whatever reason.

Did you see anything helpful in debug logs?

I didn't see an anything helpful, but I believe the job log is 49906094 and the package was apollo-graphql.

To Reproduce
I can work on this next week, if it's helpful.

Expected behavior
In a Lerna 3 monorepo, when a sub-package's dependency is updated, the root's package-lock.json gets updated as well. To be honest, the only reason I have this expectation is that it's happened in the past. This is not what I would expect during a normal npm install in a subdirectory of a Lerna monorepo without also running npm install at the root afterward (to update the package-lock.json)

@rarkins
Copy link
Collaborator

rarkins commented Mar 7, 2019

@abernix a reproduction repo would definitely be appreciated here, if you get the time next week

@rarkins rarkins added type:bug Bug fix of existing functionality help wanted Help is needed or welcomed on this issue needs-requirements manager:npm package.json files (npm/yarn/pnpm) labels Mar 7, 2019
@abernix
Copy link
Contributor Author

abernix commented Mar 7, 2019

Yup! We don't have these types of renovations on our repository very often.
Just dug around some more and saw that apollographql/apollo-server#2233 seems to have also worked as expected.

Also, the affected package in that job log was apollo-graphql.

@HonkingGoose
Copy link
Collaborator

@rarkins I think we still need a reproduction repository here to make progress on this issue?

@viceice viceice added the auto:reproduction A minimal reproduction is necessary to proceed label Mar 8, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2021

Hi there,

The Renovate team needs your help! To fix the problem, we first need to know exactly what's causing the bug. A minimal reproduction help us to pinpoint the exact cause of the bug.

To get started, please read our guide on minimal reproductions to understand what is needed.

We may close the issue if you have not provided a minimal reproduction within two weeks. If you need more time, or are stuck, please ask for help or more time in a comment.

Good luck,

The Renovate team

@github-actions
Copy link
Contributor

When a bug has been marked as needing a reproduction, it means nobody can work on it until one is provided. In cases where no reproduction is possible, or the issue creator does not have the time to reproduce, we unfortunately need to close such issues as they are non-actionable and serve no benefit by remaining open. This issue will be closed after 7 days of inactivity.

@github-actions github-actions bot added the stale label Mar 23, 2021
@github-actions
Copy link
Contributor

This bug report has been closed as we need a reproduction to work on this. If the original poster or anybody else with the same problem discovers that they can reproduce it, please create a new issue, and reference this issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto:reproduction A minimal reproduction is necessary to proceed help wanted Help is needed or welcomed on this issue manager:npm package.json files (npm/yarn/pnpm) stale status:requirements Full requirements are not yet known, so implementation should not be started type:bug Bug fix of existing functionality
Projects
None yet
Development

No branches or pull requests

4 participants