Skip to content

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

Merged
schlessera merged 1 commit intowp-cli:mainfrom
thelovekesh:update/phpcs-workflow
Nov 10, 2023
Merged

Update PHPCS workflow to add full report in CI logs#90
schlessera merged 1 commit intowp-cli:mainfrom
thelovekesh:update/phpcs-workflow

Conversation

@thelovekesh
Copy link
Member

@thelovekesh thelovekesh requested a review from a team as a code owner November 10, 2023 13:40
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
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Indicate specific PHPCS issues in CI job

3 participants