Skip to content

Conversation

@justlevine
Copy link
Collaborator

What does this implement/fix? Explain your changes.

This PR cleans up the repository by running git rm -r --cached . As a result:

  • codeception.yml has been migrated to codeception.dist.yml
  • test images have been prefixed with test-*

Does this close any currently open issues?

Any relevant logs, error output, GraphiQL screenshots, etc?

Any other comments?

Where has this been tested?

Operating System:

WordPress Version:

@justlevine justlevine requested a review from jasonbahl February 9, 2025 20:11
@justlevine justlevine added status: in review Awaiting review before merging or closing needs: reviewer response This needs the attention of a codeowner or maintainer labels Feb 9, 2025
@jasonbahl
Copy link
Collaborator

@justlevine this is breaking tests

@justlevine
Copy link
Collaborator Author

Yeah, clearly the Codeception/docker is more fragile than I thought 🧐

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit bc9563b and detected 0 issues on this pull request.

View more on Code Climate.

jasonbahl added a commit that referenced this pull request Feb 10, 2025
- Add test images to satisfy changes in #3303
jasonbahl added a commit that referenced this pull request Feb 10, 2025
- Add test images to satisfy changes in #3303
@jasonbahl jasonbahl merged commit 9e7488f into wp-graphql:develop Feb 11, 2025
35 checks passed
@justlevine justlevine deleted the chore/cleanup-git-cache branch February 11, 2025 08:08
@jasonbahl jasonbahl mentioned this pull request Feb 11, 2025
@coveralls
Copy link

Coverage Status

coverage: 83.203% (+0.009%) from 83.194%
when pulling bc9563b on justlevine:chore/cleanup-git-cache
into eb824fd on wp-graphql:develop.

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

Labels

needs: reviewer response This needs the attention of a codeowner or maintainer status: in review Awaiting review before merging or closing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants