Skip to content

Conversation

@graysonguarino
Copy link
Collaborator

@graysonguarino graysonguarino commented May 13, 2025

N/A

Issue Number:
Reference the issue this PR fixes.

Rework of #25

Description of Changes:
Provide a clear and concise explanation of what changes you made and why.

  • Add gRPC handlers for Database service
  • Add unit tests for handlers

Testing Done:
How did you test your changes? Share details like steps, tools used, or results.

  • Initial unit tests added

Terms of contribution:

By submitting this pull request, I agree that this contribution is licensed under the terms of the Apache License, Version 2.0.


Thanks for submitting your pull request! We will review it as soon as possible.

@graysonguarino graysonguarino requested a review from ecpullen May 13, 2025 18:57
@graysonguarino graysonguarino changed the base branch from main to mvp May 19, 2025 14:29
@graysonguarino graysonguarino force-pushed the server/impl-grpc-handlers-new-api branch from 0446df1 to eab0cd3 Compare May 19, 2025 20:52
@graysonguarino graysonguarino changed the title Server/impl grpc handlers new api [Server] Implement gRPC handlers with new API May 19, 2025
@graysonguarino graysonguarino changed the title [Server] Implement gRPC handlers with new API [Server] Implement gRPC handlers May 19, 2025
@graysonguarino graysonguarino marked this pull request as ready for review May 19, 2025 20:54
@hallyn-geico
Copy link
Collaborator

Run cargo fmt --all -- --check
error: 'cargo-fmt' is not installed for the toolchain '1.87.0-x8[6](https://github.com/geico/tuxtape/actions/runs/15122951183/job/42509277217?pr=26#step:4:7)_64-unknown-linux-gnu'.
To install, run `rustup component add rustfmt`

@graysonguarino
Copy link
Collaborator Author

Run cargo fmt --all -- --check
error: 'cargo-fmt' is not installed for the toolchain '1.87.0-x8[6](https://github.com/geico/tuxtape/actions/runs/15122951183/job/42509277217?pr=26#step:4:7)_64-unknown-linux-gnu'.
To install, run `rustup component add rustfmt`

@ecpullen do you want to fix the action or do you want me to take a crack at 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.

3 participants