-
Notifications
You must be signed in to change notification settings - Fork 141
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
proto, pkg, docs: update CSR docstrings #926
Conversation
Signed-off-by: William Woodruff <william@trailofbits.com>
b5f9e86
to
2e00c43
Compare
@@ -16,7 +16,7 @@ | |||
// Code generated by protoc-gen-go. DO NOT EDIT. | |||
// versions: | |||
// protoc-gen-go v1.28.1 | |||
// protoc v3.20.3 | |||
// protoc v3.21.11 |
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.
Ah, this is gonna be an issue. My recommendation is to either update
fulcio/.github/workflows/main.yml
Line 41 in d1fa542
version: '3.20.3' |
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.
My protoc
is from brew
, so downgrading might be a little painful 😅 -- I'll bump the action if that's alright.
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.
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.
Hm, it seems to think that version doesn't exist.
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.
Yeah...looks like manifestation of this issue: arduino/setup-protoc#33
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.
(It seems like protobuf changed its versioning to chop the 3.
prefix off, and this action doesn't know about that.)
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 opened arduino/setup-protoc#58 with a fix to that action, and I've temporarily updated this PR to pin my trail-of-forks
fork instead. But I assume we'll want this to be upstreamed before merging 🙂
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
The diff is still failing, for reasons that aren't clear to me...is it possible |
@woodruffw Are you using Go 1.19.4? Differences in Go versions can also cause issues |
|
Bumped my `go` version. Signed-off-by: William Woodruff <william@trailofbits.com>
Closes #863.
Signed-off-by: William Woodruff william@trailofbits.com
cc @haydentherapper