-
Notifications
You must be signed in to change notification settings - Fork 685
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
Bump sequoia-openpgp to 1.20.0 #7178
Conversation
CI needs #7179 to be happy. |
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.
CI is good, LGTM
The merge-base changed after approval.
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.
CI is good, LGTM
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.
One more time with feeling
The merge-base changed after approval.
See <https://gitlab.com/sequoia-pgp/sequoia/-/blob/openpgp/v1.20.0/openpgp/NEWS?ref_type=tags> for a list of changes, none of which should require changes on our end. Fixes #7177.
d028c02
to
56b036c
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
Status
Ready for review
Description of Changes
See https://gitlab.com/sequoia-pgp/sequoia/-/blob/openpgp/v1.20.0/openpgp/NEWS?ref_type=tags for a list of changes, none of which should require changes on our end.
Fixes #7177.
Testing
How should the reviewer test this PR?
Deployment
Any special considerations for deployment? n/a, applies equally to both
Checklist
make lint
) and tests (make test
) pass in the development container