Can !pmic.isPowerGood() requirement be removed from battery check?

[damonmaria]

I've had a Particle Electron in the field die on me. It was using this library's battery check but went all the way down to 0.3% SoC. Given the logic I presume this was because pmic.isPowerGood() was true because of the attached solar panels (the only power source)?

What is the purpose of the !pmic.isPowerGood() check? Could it be removed as it totally failed me in what I would think is a pretty standard setup (solar charging).

[damonmaria]

@rickkas7 Any thoughts on this? If you're worried about backwards compatibility then could the !pmic.isPowerGood() check be turned off with an option passed into the constructor?

[damonmaria]

@rickkas7 This has now caught us out multiple times. I'm happy to do a PR with a constructor option to check pmic.isPowerGood() or not (default being the current implementation of do the check). If you're happy with that solution then let me know. Otherwise I guess I'll have to fork this.