MetaMask / metamask-design-system

2 stars 1 forks source link

Reduce Warning Noise in `yarn test` Output to Improve Developer Experience #133

Closed georgewrmarshall closed 6 days ago

georgewrmarshall commented 1 week ago

Description

When running yarn test, there are several warnings and errors logged in the output, creating noise that can lead to "warning blindness." Developers may inadvertently ignore legitimate issues due to the sheer volume of irrelevant or minor warnings.

Why This Is Bad:

  1. Warning Blindness: The overwhelming number of warnings makes it harder to identify critical issues.
  2. Developer Experience: Excessive noise in the output detracts from a smooth development workflow.
  3. Potential Oversights: Important warnings or errors could be missed, leading to regressions or unresolved issues.
  4. CI/CD Pipeline Noise: Unnecessary warnings can obscure meaningful insights or errors during automated testing.
  5. Professionalism: Clean, clear test output reflects a project's quality and enhances onboarding for contributors and external developers.

Steps to Reproduce

  1. Clone the repository.
  2. Run yarn test from the root of the project.
  3. Observe multiple warnings, including configuration-related warnings and other noise.

Expected Behavior

Screenshots

https://github.com/user-attachments/assets/4741f36c-9003-468c-8036-ae65330e667a