Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: quiet console.error in passing api.test.ts #202

Merged
merged 1 commit into from
Dec 16, 2024
Merged

Conversation

skyqrose
Copy link
Member

Asana Task: None

This has been bugging me for a while, passing tests shouldn't print error messages. I finally spent 15 minutes to fix it.

Before:
Screenshot 2024-12-16 at 09 44 12

After:
Screenshot 2024-12-16 at 09 51 28

  • Tests:
    • ( ) Has tests
    • (x) Doesn't need tests
    • ( ) Tests deferred (with justification)
  • Product/Design sign off:
    • ( ) Okayed the plan for the feature (e.g. the design files, or the Asana task)
    • ( ) Reviewed the feature as implemented (e.g. on dev-green, or saw screenshots)
    • (x) No review needed

@skyqrose skyqrose requested a review from a team as a code owner December 16, 2024 14:55
@skyqrose skyqrose merged commit 8c4565b into main Dec 16, 2024
5 checks passed
@skyqrose skyqrose deleted the sqr-test branch December 16, 2024 20:32
@mathcolo mathcolo added the deploy-to-sandbox deploy this PR to sandbox label Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-to-sandbox deploy this PR to sandbox
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants