Skip to content

fix(graph): Fix Issue 156#167

Merged
averikitsch merged 2 commits intogoogleapis:mainfrom
amullick-git:issue156
Apr 15, 2025
Merged

fix(graph): Fix Issue 156#167
averikitsch merged 2 commits intogoogleapis:mainfrom
amullick-git:issue156

Conversation

@amullick-git
Copy link
Copy Markdown
Contributor

@amullick-git amullick-git commented Apr 12, 2025

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

BEGIN_COMMIT_OVERRIDE
fix(graph): 'verified_gql' exception
END_COMMIT_OVERRIDE

@amullick-git amullick-git requested review from a team April 12, 2025 01:18
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the googleapis/langchain-google-spanner-python API. label Apr 12, 2025
@amullick-git amullick-git requested a review from mtyin April 12, 2025 01:45
Copy link
Copy Markdown
Contributor

@mtyin mtyin left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing it.

@amullick-git
Copy link
Copy Markdown
Contributor Author

/cbrun

@averikitsch averikitsch merged commit 9940b86 into googleapis:main Apr 15, 2025
10 checks passed
@averikitsch averikitsch added the release-please:force-run To run release-please label Apr 17, 2025
@release-please release-please Bot removed the release-please:force-run To run release-please label Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the googleapis/langchain-google-spanner-python API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants