Code review comment for lp:~clissold345/ubuntu-docs/minor-changes-to-firewall-page

Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

Looks good, thanks for your work.

I made a tiny change when merging: Surrounding SSH with <app></app> seems not to be right, since SSH in this context is reasonably the protocol rather than a specific application, so I dropped those.

You can find out about the meaning of various inline elements at <http://projectmallard.org/1.0/>.

review: Approve

« Back to merge proposal