-
Notifications
You must be signed in to change notification settings - Fork 582
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
Upgrade to PG* v0.4.2 #103
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.
Super awesome. @sesmith177 can you confirm this works on Windows? We should merge in any case before too long to unblock other work.
templates/cc/register.go
Outdated
"quote": fns.quote, | ||
"inType": fns.inType, | ||
"inKey": fns.inKey, | ||
"durLit": fns.durLit, |
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.
Nit: alpha sort this list?
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.
👌
@htuch looks good on Windows. We tested by running
Also, after #96 was merged, it looks like we can simplify the powershell script here: https://github.com/lyft/protoc-gen-validate/blob/64fcb82c878efbfe3240e4f164490eefaf6b3c4b/windows/ci.ps1 to just wrap the |
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.
LGTM; @rodaine is this ready to merge?
This patch upgrades the PG* dependency of PGV to v0.4.2, and rectifies the breaking API changes from that library (lyft/protoc-gen-star#28) as well as the adopted behaviors of
protoc-gen-go
(#89).The majority of the modified files are changes in the
vendor
folder dependencies, followed by numerous changes to the BazelBUILD
files resulting from running gazelle (and the fallout thereof).Templates were updated to use the new golang context from PG*'s to properly derive package and entity names & paths, properly accounting for the interplay of the go_package options and the protoc-gen-go plugin parameters.
Also in this patch, the kitchensink tests have been removed in favor of the test harnesses. This was done in this patch to minimize how much BUILD file work was needed to get this project buildable. Future work will follow this PR to improve the development workflow on this project to be less fragile.
(woof...massive change is massive)