-
Notifications
You must be signed in to change notification settings - Fork 166
document new dependency tags in info.xml #706
Conversation
closes #687 |
developer_manual/app/info.rst
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The links needs to be in following format:
External hyperlinks, like `Python <http://www.python.org/>`_.
Beside the mentioned link issue 👍 |
c72facf
to
b5a4e87
Compare
You may want to mention how the library names and versions are determined and mention the exceptions (e.g. libxml is called xml etc, the rest is queried using phpversion('libname')) |
And maybe that http://php.net/manual/de/function.get-loaded-extensions.php is used for getting the extensions |
@Raydiation more docs added - merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In case of undefined -> In case no values are provided
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also requiremax and require i guess
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well - requiremax is not document and therefore to be ignored from my pov 🙊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, yes makes sense
👍 if the above small comments are addressed :) |
Is this for 8.0 only? Remember, master is 8.0 now. |
yes - oc8 only |
document new dependency tags in info.xml
@MorrisJobke @Raydiation @karlitschek @carlaschroder