Skip to content

Conversation

@stefanheimes
Copy link
Member

@stefanheimes stefanheimes commented Jan 26, 2026

Description

Please explain the detailed changes you made here.
Reference any issue number this pull request fixes.

Checklist

  • Read and understood the CONTRIBUTING guidelines
  • Created tests, if possible
  • All tests passing
  • Extended the README / documentation, if necessary
  • Added myself to the @authors in touched PHP files
  • Checked the changes with phpcq and introduced no new issues

@zonky2 zonky2 requested a review from discordier January 26, 2026 14:07
@zonky2 zonky2 added the bug Something isn't working label Jan 26, 2026
@zonky2 zonky2 self-requested a review January 26, 2026 14:07
@zonky2 zonky2 added this to the 2.3.* milestone Jan 26, 2026
Copy link
Contributor

@zonky2 zonky2 left a comment

Choose a reason for hiding this comment

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

LGTM

$widget
->expects(self::any())
->method('import')
->withConsecutive([Config::class, 'Config']);
Copy link
Member

Choose a reason for hiding this comment

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

shouldn't this whole block rathe be:

        $widget
            ->expects($this->once())
            ->method('import')
            ->with(Config::class, 'Config');

trusted-keys:
# composer-require-checker
- 033E5F8D801A2F8D
- B2BDAAAC6F1FDE528CD9EEC9033E5F8D801A2F8D
Copy link
Member

Choose a reason for hiding this comment

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

While these are not wrong, they point to a bug in phpcq... Apparently the cli of gnupg uses a different key fingerprint hash len than the php pgp extension.

We should fix this upstream and then remove them here again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants