Skip to content

test(useDoubleClick): remove 'vi.clearAllTimers'#227

Merged
seungrodotlee merged 2 commits into
toss:mainfrom
sukvvon:test/remove-clearAllTimers-in-useDoubleClick
May 7, 2025
Merged

test(useDoubleClick): remove 'vi.clearAllTimers'#227
seungrodotlee merged 2 commits into
toss:mainfrom
sukvvon:test/remove-clearAllTimers-in-useDoubleClick

Conversation

@sukvvon
Copy link
Copy Markdown
Contributor

@sukvvon sukvvon commented May 1, 2025

Overview

Checklist

  • Did you write the test code?
  • Have you run yarn run fix to format and lint the code and docs?
  • Have you run yarn run test:coverage to make sure there is no uncovered line?
  • Did you write the JSDoc?

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (8aecbf4) to head (3c97efc).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #227   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           33        33           
  Lines          839       839           
  Branches       254       254           
=========================================
  Hits           839       839           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sukvvon sukvvon force-pushed the test/remove-clearAllTimers-in-useDoubleClick branch from 3c97efc to 55b29fe Compare May 1, 2025 18:30
Copy link
Copy Markdown
Contributor

@seungrodotlee seungrodotlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍

@seungrodotlee seungrodotlee merged commit 99373d2 into toss:main May 7, 2025
8 checks passed
@sukvvon sukvvon deleted the test/remove-clearAllTimers-in-useDoubleClick branch May 7, 2025 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants