Matt Brown

Very useful patch thanks - looking forward to seeing this merged ASAP. Note the help text in the usage() method on line 34 also needs updating to list the optional...

What are the changes? It's a bit tedious to get a diff between what's in this pull request and what's in the branch because of the changed filenames, etc.

(can't reply directly on your comment above since I pushed the latest commit... weird) > Just wondering - wouldn't the same approach of sending a pointer to the JsonDocument have...

https://github.com/oseiler2/CO2Monitor/commit/aeaec2a844ba2bdb43334672c46107aa42a2215f doesn't quite preserve the same semantics that were intended here - specifically it doesn't take ownership of the passed status message and does not free it after it is...

If we can't use the option then we'll have to resort to parsing the schema dump and removing the tables prefixed sqlite_ manually, which doesn't sound particularly attractive to me.

I've hit this issue too so I'd like this to remain open rather than be auto-closed as stale. It's definitely a pain-point with plush.

please take another look - addressed those comments and rebased my fork to the current head.

Service connect requires an addition "256 CPU units and at least 64 MiB of memory" per proxy sidecar container which is a ridiculous amount of overhead to add just to...