Skip to content

use --hidden when querying what inputs#340

Merged
toastwaffle merged 1 commit intoplease-build:masterfrom
goddenrich:what-inputs-hidden
Feb 4, 2026
Merged

use --hidden when querying what inputs#340
toastwaffle merged 1 commit intoplease-build:masterfrom
goddenrich:what-inputs-hidden

Conversation

@goddenrich
Copy link
Contributor

@goddenrich goddenrich commented Feb 4, 2026

plz query whatinputs without --hidden attempts to convert a hidden target into a non-hidden one. It makes the assumption that if there is a target with a _{name}#{hash} then there exists a non-hidden target called name within the same package. This is not necessarily true. Given we actually can use the hidden targets running plz query --hidden is safer since we are guaranteed to have a valid target.

This change also pipes the files into stdin rather than as an argument to the command. This avoids the issue where if there are too many files a fork errors due to the length of the arguments.

@toastwaffle toastwaffle merged commit cce8f3e into please-build:master Feb 4, 2026
5 checks passed
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.

2 participants