feat: add option to not suppress error output if desired

[mpeyper]

What:

Add suppressErrorOutput option (default to true) to allow disabling of the console.error suppression we added with the error boundary fix for useEffect errors.

Why:

It was mentioned in discord that it might be causing issues for some people as its nuked their own console mocks.

How:

Just a boolean option an a if statement to skip over the call that uses filter-console.

Checklist:

• Documentation updated
• Tests
• Ready to be merged
  

I've put this up for discussion to see if:

1. this is something we want to add and support
2. is there a better way we could have handled the error logs than this

Any feedback is welcome.

[codecov]

Codecov Report

Merging #541 (86c88f3) into master (6335c4e) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #541   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           14        14           
  Lines          210       212    +2     
  Branches        23        25    +2     
=========================================
+ Hits           210       212    +2     

Impacted Files	Coverage Δ
src/dom/pure.ts	100.00% <100.00%> (ø)
src/helpers/createTestHarness.tsx	100.00% <100.00%> (ø)
src/native/pure.ts	100.00% <100.00%> (ø)
src/server/pure.ts	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6335c4e...86c88f3. Read the comment docs.

[joshuaellis]

As I mentioned in discord, I think this is probably the best way, I also definitely agree with default being true to supress. We should add it to the documentation and then it's good to go.

[mpeyper]

Closing in favour of #549