build: ignore Storybook stories in CodeCov

[ktmud]

SUMMARY

Ignore Storybook stories in CodeCov report so we don't get these unnecessary warnings:

Stories are a type of visual tests themselves and do not normally need test coverage.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A

TEST PLAN

CI

ADDITIONAL INFORMATION

• Has associated issue:
• Changes UI
• Requires DB Migration.
• Confirm DB Migration upgrade and downgrade tested.
• Introduces new feature or API
• Removes existing feature or API

[michael-s-molina]

@ktmud some stories have .jsx extension. We should also ignore those while we don't change to typescript. Thanks for this PR!

[ktmud]

@michael-s-molina These warnings only show up when you modify a file. I was hoping the next time we touch a .stories.jsx we would have converted it to .tsx already!

[michael-s-molina]

@michael-s-molina These warnings only show up when you modify a file. I was hoping the next time we touch a .stories.jsx we would have converted it to .tsx already!

I didn't know that the warnings only show when we modify.. in that case we should be fine 😁

[michael-s-molina]

Hum.. thinking a little more about it.. we have the case of executing the coverage script through npm.. in that case it will count .jsx stories. What do you think @ktmud ?

[ktmud]

Yeah, it could have an impact on coverage percentages, but should not throw the warnings. So I'm adding .jsx to the ignore list anyway.

[rusackas]

Nice catch! Thanks!