Skip to content
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

[GitHub Actions] Fix deploy-apidocs.yml for phpDocumentor usage #7833

Merged

Conversation

takano32
Copy link
Contributor

@takano32 takano32 commented Aug 17, 2023

Description
Follow-up #7832

Fix GitHub Actions Error.

e.g. https://github.com/codeigniter4/CodeIgniter4/actions/runs/5887864586/job/15967978898

Run phpDocumentor run --ansi --verbose
PHP Warning:  require(phar:///usr/local/bin/phpDocumentor/bin/phpdoc): Failed to open stream: phar error: no directory in "phar:///usr/local/bin/phpDocumentor/bin/phpdoc", must have at least phar:///usr/local/bin/phpDocumentor/bin/phpdoc/ for root directory (always use full path to a new phar) in /usr/local/bin/phpDocumentor on line 10
PHP Fatal error:  Uncaught Error: Failed opening required 'phar:///usr/local/bin/phpDocumentor/bin/phpdoc' (include_path='.:/usr/share/php') in /usr/local/bin/phpDocumentor:10
Stack trace:
#0 {main}
  thrown in /usr/local/bin/phpDocumentor on line 10
Error: Process completed with exit code 255.

Fixed according to the official phpDocumentor documentation.

see also. https://github.com/phpDocumentor/phpDocumentor

This time, I checked the operation completely in the local development environment.

Sorry and Thanks.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

Fixed according to the official phpDocumentor documentation.

see also. https://github.com/phpDocumentor/phpDocumentor
@samsonasik samsonasik requested a review from kenjis August 17, 2023 08:08
@kenjis kenjis added the github_actions Pull requests that update Github_actions code label Aug 17, 2023
@@ -49,7 +49,7 @@ jobs:

- name: Download latest phpDocumentor
working-directory: source
run: sudo phive --no-progress install --global --trust-gpg-keys 8AC0BAA79732DD42 phpDocumentor
run: phive install --force-accept-unsigned phpDocumentor
Copy link
Member

Choose a reason for hiding this comment

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

I think using --trust-gpg-keys is better. But try this PR for now.

@kenjis kenjis merged commit d8190bb into codeigniter4:develop Aug 17, 2023
@kenjis
Copy link
Member

kenjis commented Aug 17, 2023

Failed. Can you add --trust-gpg-keys again?

Error:    Needs tty to be able to confirm

	Fingerprint: F33A 0AF6 9AF7 A8B1 5017 DB52 6DA3 ACC4 991F FAE5

	phpDocumentor <[email protected]>

	Created: 2019-07-26

Import this key? [y|N] 
Error: Process completed with exit code 4.

https://github.com/codeigniter4/CodeIgniter4/actions/runs/5888811844/job/15970764918

@kenjis kenjis mentioned this pull request Aug 17, 2023
@takano32
Copy link
Contributor Author

@kenjis I see. Wait a minutes please. 🙇

@takano32
Copy link
Contributor Author

@kenjis How about this?

#7834

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update Github_actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants