Skip to content

Update LuaSandbox installation method - #5725

Open
tstarling wants to merge 1 commit into
php:masterfrom
tstarling:luasandbox-pie
Open

Update LuaSandbox installation method#5725
tstarling wants to merge 1 commit into
php:masterfrom
tstarling:luasandbox-pie

Conversation

@tstarling

Copy link
Copy Markdown
Contributor

New releases will be in PIE and Debian.

MongoDB seemed to be the nearest analogy, but what we need to say here is a lot simpler, so I pared it down to a single section. I used programlisting for the shell commands, as in MongoDB.

@tstarling
tstarling requested a review from Girgias July 30, 2026 04:39
New releases will be in PIE and Debian.
</para>

<para>
To install LuaSandbox using <link xmlns:xlink="http://www.w3.org/1999/xlink" xlink:href="https://github.com/php/pie">PIE</link>,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use the &url.pie; entity reference for href.

</programlisting>
</para>

<simpara>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we make this a note instead?

Also I think it's more consistent with the docs if we don't include "the" in the url.

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.

2 participants