script: Require zero exit code to show empty module. by patrick96 · Pull Request #2861

script: Require zero exit code to show empty module. by patrick96 · Pull Request #2861 · polybar/polybar

What type of PR is this? (check all applicable)

Refactor
Feature
Bug Fix
Optimization
Documentation Update
Other: Replace this with a description of the type of this PR

Description

The script module shows a completely empty module if the script didn't produce any output. This is intended behavior, but it doesn't make too much sense if the script also exited with a non-zero exit code.

Related Issues & Documents

Ref #2857

Documentation (check all applicable)

This PR requires changes to the Wiki documentation (describe the changes)
This PR requires changes to the documentation inside the git repo (please add them to the PR).
Does not require documentation changes

The script module docs should explicitly state that an empty output and a zero exit code will hide the script module.