Unused variable warning is polluting the Serial Console :) 10.1.3 about farmbot_os HOT 9 CLOSED

I take the blame for this one @jsimmonds2 . While implementing the filter groups / group iteration feature, I was having issues with a part of the CSVM and worked around it temporarily. Definitely a bug, but the warning does not point to a significant problem, rather it points to some corner cutting on my part. I hope I can fix this in the upcoming v3 MOVE TO block.

from farmbot_os.

the blame

"blame" . . What's that ? :-)

Can CircleCI catch the console stderr stream ? Or compile time warnings like this ?

from farmbot_os.

😛 I need to do a patch release today anyway to un-bork an unrelated issue. I will include a simple fix for this in 10.1.4.

from farmbot_os.

Ooops . . got a New one now ( only on Serial Console stderr , not SSH ) 10.1.4 commit 1c5ef14

from farmbot_os.

Hello @RickCarlino ! Has this one ( above ) been cleaned up yet ? :) Thanks !

from farmbot_os.

@jsimmonds2 It's pretty low priority given the other things that are going on this week, but I will need to go in there next week when writing the V3 MOVE block that's coming up.

Did the fix make it worse? I can remove it if that's the case.

from farmbot_os.

Thanks @RickCarlino . No problems 😅 It's not worse after the recent fix, just a different unused variable.

Rarely use the Console device, but it's the only way I know of to see the stdout and stderr streams of the BEAM.

P.S. I heard today ( in Australia ) that Chicago is recently a SARS-COV-2 "hot-spot" ! Wear a mask and look after yourself 👍

from farmbot_os.

@jsimmonds2 Found a fix while working on unrelated features. The fix is in bb077b1 and will be deployed as part of v11.0.0

from farmbot_os.

👍

from farmbot_os.