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.

Sign up for GitHub

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

Jump to bottom

feat: Grep the PHP logs on error, and show any fatals in the PR comment. #137

Merged
merged 2 commits into from
Jan 16, 2025
Merged

feat: Grep the PHP logs on error, and show any fatals in the PR comment. #137

merged 2 commits into from
Jan 16, 2025

Conversation

cafuego
Copy link
Contributor

@cafuego cafuego commented Aug 27, 2024

Refs: OPS-10706

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 27, 2024 •
edited
Loading

Build output

Composer Validate success

PHP Lint success

Docker Build success

Environment Setup success

Site Install success

PHP Code Sniffer success

Software Versions PHP 8.2.27 (cli) (built: Dec 19 2024 07:26:44) (NTS) Copyright (c) The PHP Group Zend Engine v4.2.27, Copyright (c) Zend Technologies with Zend OPcache v8.2.27, Copyright (c), by Zend Technologies with Xdebug v3.4.0, Copyright (c) 2002-2024, by Derick Rethans Composer version 2.8.4 2024-12-11 11:57:47
Drupal Logs
PHP Logs

Pusher: @lazysoundsystem, Action: pull_request, Workflow: Run tests

@lazysoundsystem
Copy link
Contributor

Thanks - I'll add this to the other repos.

@github-actions GitHub Actions
Copy link

Coverage Report

Totals Coverage

@lazysoundsystem lazysoundsystem merged commit b6074eb into main Jan 16, 2025
2 checks passed
@lazysoundsystem lazysoundsystem deleted the cafuego/ops-10706-php-errorgrep branch January 16, 2025 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lazysoundsystem lazysoundsystem lazysoundsystem approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cafuego @lazysoundsystem