Skip to content

Conversation

@nshirley
Copy link
Contributor

@nshirley nshirley commented Sep 4, 2025

Because:

  • Some file names might sneak into GCP that the tool cannot parse

This Commit:

  • Updates the error handling to just log unparsable files as warning, preventing total exit and allowing continued processing

Description

Because of the order and current handling of file parsing, if a file with a bad name exists it can kill the process and cause it to immediately exit. This commit just updates the logic to log those as warnings, allowing it to continue on.

Issues

Closes: N/A

@nshirley nshirley requested a review from b4handjr September 4, 2025 14:40
Because:
 - Some file names might sneak into GCP that the tool cannot parse

This Commit:
 - Updates the error handling to just log unparsable files as warning,
   preventing total exit and allowing continued processing
@nshirley nshirley force-pushed the chore/dont-fail-on-unprased-file branch from 4636c10 to 50da258 Compare September 4, 2025 14:43
Copy link
Contributor

@b4handjr b4handjr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

@nshirley nshirley added this pull request to the merge queue Sep 4, 2025
Merged via the queue into main with commit 65674d8 Sep 4, 2025
2 checks passed
@nshirley nshirley deleted the chore/dont-fail-on-unprased-file branch September 4, 2025 15:13
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.

3 participants