Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(manager/npm): improve handling of grouped updates #31326

Merged
merged 3 commits into from
Sep 13, 2024

Conversation

RahulGautamSingh
Copy link
Collaborator

@RahulGautamSingh RahulGautamSingh commented Sep 11, 2024

Changes

  • Before executing corepack update, write the existing files to disk so we do not lose them

Context

Closes: #31325

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 1 2
  • Both unit tests + ran on a real repository

@RahulGautamSingh RahulGautamSingh marked this pull request as draft September 11, 2024 07:55
@RahulGautamSingh RahulGautamSingh marked this pull request as ready for review September 13, 2024 09:06
@RahulGautamSingh RahulGautamSingh marked this pull request as draft September 13, 2024 09:12
@RahulGautamSingh RahulGautamSingh marked this pull request as ready for review September 13, 2024 09:12
@rarkins rarkins added this pull request to the merge queue Sep 13, 2024
Merged via the queue into renovatebot:main with commit 88b0348 Sep 13, 2024
38 checks passed
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 38.77.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

npm packageManager updates lose other npm updates if grouped together
4 participants