Add server_number options to firewall modules

[felixfontein]

SUMMARY

Identifying the server for firewalls by server main IP is deprecated (https://robot.hetzner.com/doc/webservice/en.html#get-firewall-server-id at the very bottom). This PR adds a server_number option that allows to identify the server by its identifier.

ISSUE TYPE

• Feature Pull Request

COMPONENT NAME

firewall
firewall_info

[github-actions]

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and the docs are now incorporated into main:
https://ansible-collections.github.io/community.hrobot/branch/main

[codecov]

Codecov Report

Merging #77 (36920ad) into main (6640769) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #77   +/-   ##
=======================================
  Coverage   98.30%   98.30%           
=======================================
  Files          32       32           
  Lines        2302     2302           
  Branches      230      230           
=======================================
  Hits         2263     2263           
  Misses         18       18           
  Partials       21       21           

Flag	Coverage Δ
sanity	27.41% <0.00%> (ø)
units	98.16% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
plugins/modules/firewall.py	100.00% <100.00%> (ø)
plugins/modules/firewall_info.py	100.00% <100.00%> (ø)
tests/unit/plugins/modules/test_firewall.py	100.00% <100.00%> (ø)
tests/unit/plugins/modules/test_firewall_info.py	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[felixfontein]

@filviu you might also be interested in this PR, though please don't test it before I rebased after merging the other one - otherwise it will remove output rules as well ;-)

[maxamillion]

FWIW the changes seem reasonable and the code is good. I don't understand the implications to the collection functionality though.

[felixfontein]

Rebased after #76 has been merged.

[felixfontein]

@maxamillion thanks for reviewing this!