Skip to content

Upgrade to guides#202

Closed
javiereguiluz wants to merge 6 commits intosymfony-tools:mainfrom
javiereguiluz:upgrade_to_guides
Closed

Upgrade to guides#202
javiereguiluz wants to merge 6 commits intosymfony-tools:mainfrom
javiereguiluz:upgrade_to_guides

Conversation

@javiereguiluz
Copy link
Collaborator

@javiereguiluz javiereguiluz commented Mar 1, 2026

Disclaimer: this is 100% AI code and 0% human code. I haven't done anything myself and I haven't even looked at phpDocumentor Guides code.


I asked AI to migrate from Doctrine RST to phpDocumentor Guides. I don't know if this is correct, but all tests pass with minimal cosmetic changes (white spaces, new lines, etc.). So, I assume the changes are technically correct.

I'm not saying that we should use this PR. I know that @wouterj has been working on this migration for some time. So, I think we should let Wouter decide what to do here:

  • Don't merge this but take some ideas from it for the manual migration PR
  • Merge this but make some changes based on the ideas of the manual migration PR
  • other?

Since this PR took zero human effort, we can close it or do anything we want about it. Thanks!

@wouterj
Copy link
Contributor

wouterj commented Mar 1, 2026

Honestly, this part of the work is all finished (#200). What is needed is resolving that pull request's conflicts and integrating it in the symfony.com repository.

For this we need to contribute better asset management to phpDocumentor Guides, and we need to investigate if phpDocumentor/guides#1277 is a real problem for us or not (toctree not in same order as Sphinx/RST parser).

I think if you want to spend time on this, time is better spend on those parts :). But if you just want to experiment with AI: go ahead ;)

@javiereguiluz
Copy link
Collaborator Author

OK, so let's close this in favor of #200. If you can, please turn it into a real PR (it's a draft right now) so we can start reviewing it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants