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

Update PHPCS workflow to add full report in CI logs #90

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

thelovekesh
Copy link
Member

@thelovekesh thelovekesh requested a review from a team as a code owner November 10, 2023 13:40
@@ -9,4 +9,4 @@ on:

jobs:
code-quality:
uses: wp-cli/.github/.github/workflows/reusable-code-quality.yml@main
uses: thelovekesh/wp-cli-dot_github/.github/workflows/reusable-code-quality.yml@update/phpcs-workflow
Copy link
Member

Choose a reason for hiding this comment

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

This needs to be reverted.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I pushed it for testing.

Copy link
Member Author

Choose a reason for hiding this comment

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

Reverted 👍🏼

@schlessera schlessera merged commit c2950a5 into wp-cli:main Nov 10, 2023
14 checks passed
@thelovekesh thelovekesh deleted the update/phpcs-workflow branch November 10, 2023 15:43
@danielbachhuber
Copy link
Member

Nice work on this, @thelovekesh !

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