Skip to content
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

tools/j2y2j: errors #402

Merged
merged 2 commits into from
Aug 25, 2023
Merged

tools/j2y2j: errors #402

merged 2 commits into from
Aug 25, 2023

Conversation

alrs
Copy link
Contributor

@alrs alrs commented Aug 15, 2023

This picks up a dropped err variable in tools/j2y2j and more thoroughly logs the cause of error conditions.

tools/j2y2j: log clearer error messages
@alrs alrs requested a review from a team as a code owner August 15, 2023 10:19
@timburks
Copy link
Contributor

@alrs thanks!

@timburks timburks merged commit 65eab73 into google:main Aug 25, 2023
2 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