FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Move PHPT documentation to docs by cmb69 · Pull Request #15939 · php/php-src · GitHub

/ php-src Public

Move PHPT documentation to docs - #15939

Closed
cmb69 wants to merge 2 commits into
php:masterfrom
cmb69:cmb/phpt-docs
Closed

Move PHPT documentation to docs#15939
cmb69 wants to merge 2 commits into
php:masterfrom
cmb69:cmb/phpt-docs

Conversation

cmb69 commented Sep 17, 2024

Copy link
Copy Markdown
Member

This was so far hosted on qa.php.net, but since there are plans to retire that site, we move the relevant documentation about PHPTs to php-src/docs.


Note that these are only a first few sections of https://qa.php.net/phpt_details.php, so the format can be determined/discussed before porting the whole documentation (including https://qa.php.net/write-test.php).

cc @derickr

This was so far hosted on qa.php.net, but since there are plans to
retire that site, we move the relevant documentation about PHPTs to
php-src/docs.

Copy link
Copy Markdown
Member

Unfortunately, rstfmt doesn't support custom directives 😞 There were multiple reports about it but I didn't find a workaround. If we require them at some point, I don't object to dropping the tool. It's more important to have good documentation than nicely formatted one.

cmb69 commented Sep 17, 2024

Copy link
Copy Markdown
Member Author

Unfortunately, rstfmt doesn't support custom directives

No problem; I've already updated the PR. It might be a good idea to add the rstfmt check to make.bat, and maybe to also provide a make.sh doing the same.

I'm going to also provide a PR for an MD alternative; then we can see what we like better.

derickr commented Apr 24, 2025

Copy link
Copy Markdown
Member

I have a much more comprehensive PR at #18409

derickr commented Apr 25, 2025

Copy link
Copy Markdown
Member

My large PR has now been merged, so I would suggest we close this one.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL