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

chore: fix some bad conversions #9703

Merged
merged 4 commits into from
Apr 4, 2024

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Apr 4, 2024

Be more defensive with type conversions.

BEGIN_NESTED_COMMIT
fix(pubsublite): fix int conversion
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
fix(spanner): fix uint8 conversion
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
fix(auth): fix uint32 conversion
END_NESTED_COMMIT

Fixes: #9704

Be more defensive with type conversions.

BEGIN_NESTED_COMMIT
fix(pubsublite): fix int conversion
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
fix(spanner): fix uint8 conversion
END_NESTED_COMMIT
@codyoss codyoss requested review from a team and shollyman as code owners April 4, 2024 14:50
@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Apr 4, 2024
@gcf-merge-on-green gcf-merge-on-green bot merged commit 9221c7f into googleapis:main Apr 4, 2024
10 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Apr 4, 2024
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.

Fix code scanning alert
2 participants