-
Notifications
You must be signed in to change notification settings - Fork 512
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
Carry v0.4.3 master bump #1072
Carry v0.4.3 master bump #1072
Conversation
-ldflags "-w ${CTIMEVAR} ${LDFLAGS}" \ | ||
./cmd/notary; | ||
set +x; | ||
done |
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.
Non-blocking nitpicky-ness - could we add a newline afterward?
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.
done!
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!
c549cd6
to
a30769d
Compare
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
@riyazdf can you rebase then assuming tests pass we're good to merge. |
Signed-off-by: Riyaz Faizullabhoy <[email protected]>
Signed-off-by: Riyaz Faizullabhoy <[email protected]>
… slash Also updates a test to fix the expected vs actual values. Signed-off-by: Riyaz Faizullabhoy <[email protected]>
Signed-off-by: Riyaz Faizullabhoy <[email protected]>
Signed-off-by: Riyaz Faizullabhoy <[email protected]>
a30769d
to
4cf0688
Compare
green after rebase! Merging. |
Carries patches from v0.4.3 back to master - the delegation key import panic fix and netgo flag for server flag panic.
There was some messiness because of repeat lint commits that already made it to master, so I removed those in this PR.