jamesoff / simplemonitor

A Python-based network and host monitor
https://simplemonitor.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
404 stars 165 forks source link

Add requirement for get_params in the documentation on Monitor creation #1389

Closed wsw70 closed 4 months ago

wsw70 commented 4 months ago

When writing a Monitor I realized that the get_params() method must be added in order for Loggers to know what to log. It was missing from the HOWTO Write Your Own Monitor.

EDIT: ah crap - I seriously messed that one, I just wanted to share commit @wsw70 added get_params to monitor cration docs

The others should not be there (this a new Monitor).

Sorry, I am not good with the Github PR process - and do not know how to retract this PR

EDIT 2: looks like I managed to move it to a "draft" status and will look from there