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

ci: implement phpcs, following the wp-graphql implementation #57

Merged
merged 3 commits into from
Feb 28, 2024

Conversation

jasonbahl
Copy link
Collaborator

@jasonbahl jasonbahl commented Feb 28, 2024

  • adds composer dev dependencies and scripts for phpcs and phpcbf
  • adds phpcs.xml.dist from wp-graphql with changes per existing files in this plugin
  • updates plugin header
  • updates formatting to be phpcs compliant
  • adds tests/_output directory for caching output of test and scripts like phpcs
  • updates composer.lock file
  • adds github workflow to check the code standards when PRs are opened

closes #56

- adds phpcs.xml.dist from wp-graphql with changes per existing files in this plugin
- updates plugin header
- updates formatting to be phpcs compliant
- adds tests/_output directory for caching output of test and scripts like phpcs
- updates composer.lock file
- adds github workflow to check the code standards when PRs are opened
@jasonbahl jasonbahl self-assigned this Feb 28, 2024
jasonbahl and others added 2 commits February 28, 2024 16:21
Co-authored-by: Joe Fusco <josephfusco@users.noreply.github.com>
Co-authored-by: Joe Fusco <josephfusco@users.noreply.github.com>
@josephfusco josephfusco merged commit 79097dc into main Feb 28, 2024
4 checks passed
@jasonbahl jasonbahl deleted the ci/phpcs branch May 17, 2024 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement phpcs
2 participants