Skip to content

Conversation

@btea
Copy link
Member

@btea btea commented Mar 8, 2025

close #9240

@btea btea requested a review from zkochan as a code owner March 8, 2025 12:13
@zkochan
Copy link
Member

zkochan commented Mar 8, 2025

What if the ignored package was uninstalled?

@btea
Copy link
Member Author

btea commented Mar 8, 2025

I tested it, and it seems that the information is correctly removed from .modules.yaml when the package is uninstalled.

@cghiurea
Copy link

cghiurea commented Mar 9, 2025

close #9240

@btea Thank you.

@c3cris
Copy link

c3cris commented Mar 15, 2025

What if the ignored package was uninstalled?

Any update on this?

@zkochan
Copy link
Member

zkochan commented Mar 19, 2025

Now it duplicates the same fix at @pnpm/headless but I am not sure if it could be refactored. Maybe by moving the fix to @pnpm/modules-yaml.

@zkochan zkochan merged commit bd94ce0 into pnpm:main Mar 19, 2025
8 checks passed
zkochan added a commit that referenced this pull request Mar 19, 2025
)

* fix: keep the `ignoredBuilds` field when installing different dependencies

* fix: print warning about ignored builds

---------

close #9240

Co-authored-by: Zoltan Kochan <[email protected]>
@btea
Copy link
Member Author

btea commented Mar 19, 2025

I agree that repeated logic can be moved together to prevent omissions in subsequent similar operations.

@btea btea deleted the fix/install-new-pkg-should-keep-ignoredBuilds branch March 19, 2025 00:18
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.

Persistent pending approve-builds

4 participants