Skip to content

[skip-revcheck] Convert class markup to be compatible with DocBook 5.2 - #3415

Closed
Girgias wants to merge 2 commits into
php:masterfrom
Girgias:docbook52-conversion
Closed

[skip-revcheck] Convert class markup to be compatible with DocBook 5.2#3415
Girgias wants to merge 2 commits into
php:masterfrom
Girgias:docbook52-conversion

Conversation

@Girgias

@Girgias Girgias commented May 27, 2024

Copy link
Copy Markdown
Member

Conversion performed with the following script: https://gist.github.com/Girgias/8fc473b7fda9007407ca85df32ad1b1d

@haszi do you see any issue in regard to rendering/indexing/search?

@Girgias

Girgias commented May 27, 2024

Copy link
Copy Markdown
Member Author

@kocsismate just to inform you about changes to doc format that probably need to be applied gen_stubs.php

@haszi

haszi commented May 27, 2024

Copy link
Copy Markdown
Contributor

There is an issue with classes not being linked to but this is most likely an issue in PhD.

@TimWolla TimWolla left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ack for reference/random

@ndossche

Copy link
Copy Markdown
Member

I don't know docbook but seems fine for dom and xml stuff.

@kocsismate

Copy link
Copy Markdown
Member

Filed php/php-src#14351 for gen_stub.php

@haszi

haszi commented Jun 2, 2024

Copy link
Copy Markdown
Contributor

The PhD PR for rendering the new markup.

@kocsismate

Copy link
Copy Markdown
Member

@Girgias Is it now mergable?

@Girgias

Girgias commented Jun 14, 2024

Copy link
Copy Markdown
Member Author

This should be yes, I'll merge them manually while updating the translations.

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.

5 participants