Skip to content

feat: support updateLockFiles for all managers #35993

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

Open
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

reduckted
Copy link
Contributor

@reduckted reduckted commented May 17, 2025

Changes

This changes the existing updateLockFiles option to apply to all managers. Previously that option only applied to the npm manager.

Context

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@reduckted reduckted requested a review from viceice May 18, 2025 12:34
@reduckted reduckted requested a review from viceice May 21, 2025 11:49
@reduckted reduckted requested a review from rarkins May 25, 2025 05:05
viceice
viceice previously approved these changes May 25, 2025
@reduckted reduckted changed the title feat: Add option skipArtifactUpdating feat: support updateLockFiles for all managers May 27, 2025
@@ -982,7 +982,6 @@ const options: RenovateOptions[] = [
description: 'Set to `false` to disable lock file updating.',
type: 'boolean',
default: true,
supportedManagers: ['npm'],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should maintain a list which managers support lockfiles in general

@reduckted reduckted requested a review from viceice May 29, 2025 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New option: skipArtifactUpdating
4 participants