Skip to content

Defend against missing files in the hashing logic#907

Open
dobesv wants to merge 3 commits into
microsoft:mainfrom
dobesv:hash-file-missing-fixup
Open

Defend against missing files in the hashing logic#907
dobesv wants to merge 3 commits into
microsoft:mainfrom
dobesv:hash-file-missing-fixup

Conversation

@dobesv
Copy link
Copy Markdown
Contributor

@dobesv dobesv commented May 14, 2025

If a file is missing while calculating file hashes, use stub values instead of exiting with an error.

I added this because I was occasionally getting errors from this code about the file not being there and lage would exit, I think this may be caused by my changes in #867 , which allows inputs to specify exact files rather than globs so there could be a file listed as an input that doesn't exist at the time of the build.

dobesv and others added 2 commits May 18, 2026 12:46
If a file is missing while calculating file hashes, use stub values instead of exiting with an error.
@dobesv dobesv force-pushed the hash-file-missing-fixup branch from f2cc661 to 50487ed Compare May 18, 2026 19:46
@dobesv
Copy link
Copy Markdown
Contributor Author

dobesv commented May 18, 2026

@ecraig12345 I see you approved this but did not merge it - am I supposed to merge it after ? I don't think I am able. Or are you waiting for someone else to merge it?

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