-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add php docs build script #22432
Add php docs build script #22432
Conversation
Signed-off-by: Julius Härtl <jus@bitgrid.net>
This looks amazing! |
@juliushaertl could you make sure that OC things don't get indexed? We don't want people using that ;) |
Yep, I'll have another look. Seems mainly caused by invalid
|
ah then we just fix those :) |
https://nextcloud-server.netlify.app/packages/ocp-appframeworkanyapplicationmustinheritthiscall 😂 (Nothing wrong with it. I just laughed about the long package name) |
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Pushed a fix. As a general thing I think we could just drop all https://docs.phpdoc.org/latest/references/phpdoc/tags/package.html
Since we don't introduce any different structure removing them will make the docs output a lot cleaner with out having duplicated hierarchies. |
I think we only have them because phpstorm adds those when you create a new class |
server/lib/public/App/ManagerEvent.php Line 32 in cb2695d
Mind to remove? To fix App and APP. |
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Pushed a commit to remove all the |
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 32344: failureacceptance-app-files
Show full log
|
For nextcloud/documentation#1782
Adds a build script to build php api documentation on netlify using phpDocumentor. A preview can be found at https://nextcloud-server.netlify.app/