Jim P

We used to have a switch like this to kill states for a gateway (`-G`) several years ago but there was some reason we removed it. It may have been...

It doesn't look like the concerns I raised about the function names were addressed yet, they're still named generically (`get_cpu_stats()`, `get_cpu_temperature()`) but they should be prefixed with `lcdproc_` at least.

To effectively track changes and bug/feature requests, we require an entry on the [pfSense Redmine issue tracker](https://redmine.pfsense.org) associated with every pull request, and likewise the Redmine entry should also have...

Looking at https://nmap.org they seem to prefer `Nmap` or `nmap`, I don't see anywhere they capitalize it as `NMap`. We should standardize around their preferred format, so using `Nmap` is...

This seems like it's incomplete and potentially confusing to users. Seems to me that there should be a list of alternate chains somewhere, at least for the default choices, like...

Yes, that should cover the concerns I had so far. Also, make sure the chain selection drop-down has a "Default" option that is selected by default and does not add...

Added manually (and it needed additional fixes), closing PR.

To effectively track changes and bug/feature requests, we require an entry on the [pfSense Redmine issue tracker](https://redmine.pfsense.org) associated with every pull request, and likewise the Redmine entry should also have...

As a part of this pull request, you must increase PORTVERSION or add a PORTREVISION line in the package Makefile. Without this version increase, the package will not be rebuilt...

> There is an issue reported by portlint but it existed before my edits `FATAL: Makefile: [1]: $FreeBSD$ is deprecated in Git.` > > Shall I remove that line in...