Skip to content

refector: Drop usage of xml_set_object()#152

Merged
k00ni merged 2 commits intosemsol:masterfrom
Girgias:remove-set-xml-usage
Oct 20, 2023
Merged

refector: Drop usage of xml_set_object()#152
k00ni merged 2 commits intosemsol:masterfrom
Girgias:remove-set-xml-usage

Conversation

@Girgias
Copy link
Copy Markdown
Contributor

@Girgias Girgias commented Oct 7, 2023

Use proper callables instead.

Related to: php/php-src#12340

Use proper callables instead.

Related to: php/php-src#12340
@Girgias
Copy link
Copy Markdown
Contributor Author

Girgias commented Oct 9, 2023

PHP_CS_FIXER warning is unrelated to this PR and a pre-existing issue.

@k00ni
Copy link
Copy Markdown
Collaborator

k00ni commented Oct 16, 2023

Thank you @Girgias. I am super busy currently, but I will look into that soonish. PR looks good at first glance, but I may have to fix our CI before a merge because only coding style check ran.

@k00ni k00ni merged commit 3e08bc2 into semsol:master Oct 20, 2023
@k00ni
Copy link
Copy Markdown
Collaborator

k00ni commented Oct 20, 2023

Thank you @Girgias

@Girgias Girgias deleted the remove-set-xml-usage branch October 20, 2023 10:54
@Girgias
Copy link
Copy Markdown
Contributor Author

Girgias commented Oct 20, 2023

Glad to help :D

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