fix module docs; set target to default for absent, disabled; fix unit tests

linux-system-roles / firewall

Configure firewalld and system-config-firewall

https://linux-system-roles.github.io/firewall/

GNU General Public License v2.0

58 stars 32 forks source link

fix module docs; set target to default for absent, disabled; fix unit tests #42

Closed richm closed 3 years ago

richm commented 3 years ago

Fix module docs - minor updates When setting target with state: absent or state: disabled, set value to default Fix some failing unit tests

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 2cb930dadad9717292770cf7302c9faed3eb0948 into 8866b9ec92a9e1bb865d7617f6db2cf72156fb35 - view on LGTM.com

new alerts:

1 for Unused local variable

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert and fixes 1 when merging 29309b92e7ed69a6960381a7891aa459d027003e into 520f1abf10b6dc1e4e4ad164b0b69ec614771a6d - view on LGTM.com

new alerts:

1 for Syntax error

fixed alerts:

1 for Unused local variable