Skip to content

Setup the signing of the phar file. Needed for Phive #397#663

Merged
AJenbo merged 2 commits intopdepend:masterfrom
tvbeek:phar_file_signing
Apr 15, 2023
Merged

Setup the signing of the phar file. Needed for Phive #397#663
AJenbo merged 2 commits intopdepend:masterfrom
tvbeek:phar_file_signing

Conversation

@tvbeek
Copy link
Member

@tvbeek tvbeek commented Apr 13, 2023

Type: feature
Issue: #397
Breaking change: no

To be honest, I have use https://github.com/phpDocumentor/phar-ga as an example for signing the phar file 😃

@ravage84 If I'm correct you are currently the only one that can manage the secrets. Can you create/verify the secrets? I'm not sure if the key file has a PASSPHRASE.

@tvbeek tvbeek requested a review from ravage84 April 13, 2023 20:42
@tvbeek tvbeek self-assigned this Apr 13, 2023
AJenbo
AJenbo previously approved these changes Apr 13, 2023
@codecov-commenter
Copy link

codecov-commenter commented Apr 14, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (2627cab) 82.09% compared to head (e70efc7) 82.09%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #663   +/-   ##
=========================================
  Coverage     82.09%   82.09%           
  Complexity     3440     3440           
=========================================
  Files           154      154           
  Lines          9466     9466           
=========================================
  Hits           7771     7771           
  Misses         1695     1695           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@AJenbo AJenbo merged commit 3163be6 into pdepend:master Apr 15, 2023
@ravage84 ravage84 added this to the 2.x (unspecific) milestone May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants