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
PBF support for /expansion
#4614
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have absolutely nothing to add here, good job:)
Well, other than extending the expansion test to prove the PBF stuff.
also don't feel like you'd have to wait for #4601. it's a bit unfortunate, but not a big deal to do it after this PR was merged to master. |
I thought there was no necessity for new tests since the protobuf representation is used anyway for the default JSON serialization. |
Right, no detailed tests, but there should somewhere be a pbf output test for expansion IMO. |
…to cb-expansion-pbf
|
||
using namespace valhalla; | ||
|
||
void writeExpansionProgress(Expansion* expansion, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
snake_case this
Tasklist