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
[Call-by-name] Investigate why so many intrinsic migrations are missing (e.g. Digest code) #20745
Comments
Trying to investigate why we have no migration data for the intrinsics. As an example, there is no mention of
The associated intrinsic code appears to be present:
Ditto for the associated pants/src/python/pants/engine/fs.py Line 333 in d2a9a46
So, I guess: Why is there no migration plan for intrinsics (it's more than just the filesystem, for example |
This feels like the missing piece of the puzzle. pants/src/rust/engine/src/tasks.rs Lines 185 to 196 in 572627f
|
This will end up being a larger ticket, as more of the codebase is converted to call-by-name syntax.
So instead of trying to bulk fill it in the existing PR and leave that rotting, better to cut that off more cleanly and move investigations to a separate ticket
The text was updated successfully, but these errors were encountered: