Skip to content

Conversation

@afilina
Copy link
Contributor

@afilina afilina commented Oct 15, 2025

Although the vast majority of an old PR #537 was merged, the suggestions were not addressed. Per the original author's request, I'm addressing these suggestions in this new PR.

Since examples are numbered, and we don't want that here. Since informalexample doesn't accept a title, moped it to a comment to keep the context.
@afilina afilina changed the title Improve soap ctor Improve SOAP constructor docs Oct 15, 2025
Copy link
Member

@ndossche ndossche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have commit rights?

@afilina
Copy link
Contributor Author

afilina commented Oct 15, 2025

@nielsdos I do.

Now that the acronym was merged in doc-base.
@afilina
Copy link
Contributor Author

afilina commented Oct 21, 2025

@nielsdos Now that the doc-base was merged with the WSDL acronym, I was able to address the last comment. Please merge if it's good. Thanks.

Copy link
Member

@ndossche ndossche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@ndossche ndossche merged commit f309e78 into php:master Oct 21, 2025
2 checks passed
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.

2 participants