- 1 for Unused variable, import, function or class
We need a way to reset local module variables like this, otherwise in tests they keep state if the test initializes them. I recommend adding a
resetFlair method that is called after each test.
Overall looks great. I made a couple comments.
Can we update this to the pattern where the attributes are not included unless
true to save space in the JSON?
Can be done by using
true as the method value then checking the object in the
Thanks @eviltrout I pushed those fixes.
Looks good thanks!