alfsb

Results 106 comments of alfsb

Remove the three duplicated entities from `language-snippets.ent`? Meanwhile, this is waiting for idempotent to get merged (so regression tests get a little less random), but it's in good enough shape...

> Would changing the entities generated by `file-entities.php` to Process Instructions that are transformed into an XInclude (which takes a system path) fix the issue? > > Yes this would...

> Would changing the entities generated by `file-entities.php` to Process Instructions that are transformed into an XInclude (which takes a system path) fix the issue? In the end, a simple...

Conflict resolved. But the main question remains. Do the languages ​​manuals want to split `language-snippets.ent` and/or rework the Entity DTD files into these XML Entity bundles?

This PR enables splitting `language-snippets.ent` as individual entity level files, not exactly extension level bundles. For extension related entities, there is the alternative of XInclude by `xml:id`. Yet, the reason...

> Entity level is already an improvement, so I am fine with the idea and approach. :) This will be the my next big project, then, in February. There is...

Now that all my other big projects are mostly concluded, I will start working on this again. After https://github.com/php/doc-base/pull/237, I'm now way more confident that XInclude by `xml:id` is working...

> Well, isn't that expected behavior? I would argue is not expected behavior that CI tests test a version of code that will never exist. https://github.com/php/doc-base/pull/205#issuecomment-2541033072 Taking a branch from...

> I was not particularly referring to doc-*, but rather to "all" GH repos using the checkout action (not only those of the php organization). That is something I asked...

Checking out the `checkout` action [PR and issues](https://github.com/actions/checkout), there various discussions for and against [merge commit checkout](https://github.com/actions/checkout?tab=readme-ov-file#Checkout-pull-request-HEAD-commit-instead-of-merge-commit), with the implication that is the merge is the default options. But experience...