Skip to content

BrainMonkey/YoastTestCase: rename parameters for mocked functions#53

Merged
jrfnl merged 2 commits intodevelopfrom
JRF/QA/rename-parameters-for-mocked-functions
May 1, 2023
Merged

BrainMonkey/YoastTestCase: rename parameters for mocked functions#53
jrfnl merged 2 commits intodevelopfrom
JRF/QA/rename-parameters-for-mocked-functions

Conversation

@jrfnl
Copy link
Copy Markdown
Collaborator

@jrfnl jrfnl commented May 1, 2023

BrainMonkey/YoastTestCase: rename parameter for mocked function

... as WP Core has renamed the parameter as part of the efforts to become compatible with PHP 8.0.

Ref: WordPress/wordpress-develop@51fae8b

BrainMonkey/YoastTestCase: rename parameter for mocked function

... to be in line with the parameter name as used by WP Core for optimal compatibility with PHP 8.0.

Ref: https://developer.wordpress.org/reference/functions/wp_parse_args/

jrfnl added 2 commits May 1, 2023 21:09
... as WP Core has renamed the parameter as part of the efforts to become compatible with PHP 8.0.

Ref: WordPress/wordpress-develop@51fae8b
... to be in line with the parameter name as used by WP Core for optimal compatibility with PHP 8.0.

Ref: https://developer.wordpress.org/reference/functions/wp_parse_args/
@jrfnl jrfnl added this to the 1.x Next Release milestone May 1, 2023
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage: 98.81%. Remained the same when pulling 6f76e88 on JRF/QA/rename-parameters-for-mocked-functions into 7da5034 on develop.

1 similar comment
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage: 98.81%. Remained the same when pulling 6f76e88 on JRF/QA/rename-parameters-for-mocked-functions into 7da5034 on develop.

@jrfnl
Copy link
Copy Markdown
Collaborator Author

jrfnl commented May 1, 2023

FYI: similar changes have been pulled to BrainMonkey itself and are expected to be included in the next release.
Ref: Brain-WP/BrainMonkey#136

@jrfnl jrfnl merged commit fb95b46 into develop May 1, 2023
@jrfnl jrfnl deleted the JRF/QA/rename-parameters-for-mocked-functions branch May 1, 2023 19:20
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.

2 participants