Skip to content

Conversation

@justlevine
Copy link
Collaborator

What does this implement/fix? Explain your changes.

Somehow, I shipped the get_extensions() function as get_extenions() (missing s). This fixes that super awkward mistake.

Considering this a regression.

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: Ubuntu 24.04 (wsl2 + ddev + php 8.2.26)

WordPress Version: 6.7.1

@justlevine justlevine requested a review from jasonbahl February 8, 2025 17:26
@qlty-cloud-legacy
Copy link

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

View more on Code Climate.

@justlevine justlevine added status: in review Awaiting review before merging or closing needs: reviewer response This needs the attention of a codeowner or maintainer regression Bug that causes a regression to a previously working feature labels Feb 8, 2025
@coveralls
Copy link

Coverage Status

coverage: 83.194%. remained the same
when pulling c705e48 on justlevine:fix/get_extensions
into eb824fd on wp-graphql:develop.

@jasonbahl jasonbahl merged commit 77dd979 into wp-graphql:develop Feb 10, 2025
36 checks passed
@justlevine justlevine deleted the fix/get_extensions branch February 10, 2025 21:51
@jasonbahl jasonbahl mentioned this pull request Feb 11, 2025
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 regression Bug that causes a regression to a previously working feature status: in review Awaiting review before merging or closing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants