Skip to content

Conversation

@nejch
Copy link
Member

@nejch nejch commented Apr 22, 2020

The commit signature test was failing against nightly after some changes to commit signatures (https://gitlab.com/gitlab-org/gitlab/-/merge_requests/27327). This fixes the test and some minor docs, since with 12.10 released we'll be getting failed builds without this (it was just a silly assert for the 404 :D).

I only left the unit test as is since the GPG response is one of the possible options.

@nejch nejch force-pushed the chore/signature-gpg-x509 branch from 4279aee to dc382fe Compare April 22, 2020 21:31
@nejch
Copy link
Member Author

nejch commented Apr 22, 2020

Hm, to get this to pass I had to also remove an outdated token test for the CLI.

@nejch nejch requested a review from max-wittig April 22, 2020 22:39
Copy link
Member

@max-wittig max-wittig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for keeping this up to date!

@max-wittig max-wittig merged commit 5a75310 into master Apr 23, 2020
@nejch nejch deleted the chore/signature-gpg-x509 branch May 30, 2021 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants