Skip to content

[Release] Upgrade deps and add tests - #340

Merged
SMAKSS merged 2 commits into
masterfrom
develop
Feb 1, 2026
Merged

[Release] Upgrade deps and add tests#340
SMAKSS merged 2 commits into
masterfrom
develop

Conversation

@SMAKSS

Copy link
Copy Markdown
Owner

No description provided.

CopilotAI review requested due to automatic review settings February 1, 2026 19:14

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades project dependencies and introduces comprehensive test coverage for the useDetectScroll hook. It includes dependency updates to Vite 7.3.1 and Vitest 2.1.9, along with a complete test suite.

Changes:

  • Added Vitest testing framework with coverage configuration
  • Renamed types.ts to useDetectScroll.types.ts and updated all import references
  • Upgraded Vite from 6.4.1 to 7.3.1 and related dependencies

Reviewed changes

Copilot reviewed 8 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
vitest.config.tsAdds Vitest configuration with V8 coverage provider and appropriate exclusions
src/useDetectScroll.test.tsIntroduces comprehensive test suite covering scroll detection scenarios
src/useDetectScroll.tsUpdates import path from ./types to ./useDetectScroll.types
src/index.tsUpdates export path from ./types to ./useDetectScroll.types
package.jsonAdds testing dependencies and upgrades Vite to version 7.3.1
playground/package.jsonUpgrades Vite and React plugin dependencies
.vscode/settings.jsonAdds VSCode workspace configuration for ESLint and TypeScript
.prettierignoreAdds coverage directory and pnpm-lock.yaml to ignored files

@SMAKSS
SMAKSS merged commit fa7ae05 into masterFeb 1, 2026
5 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 4.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@SMAKSS