Skip to content

test: remove the use of ts import from tests to support node 20#9

Merged
vikkrantxx7 merged 1 commit intomainfrom
bugfix/fix-ts-file-import-in-tests
Nov 10, 2025
Merged

test: remove the use of ts import from tests to support node 20#9
vikkrantxx7 merged 1 commit intomainfrom
bugfix/fix-ts-file-import-in-tests

Conversation

@vikkrantxx7
Copy link
Owner

📖 Overview

This fixes the ts import in tests which are not supported by node 20 and was earlier introduced in dev using node 22.

🎯 Type of Change

  • 🐛 Bug fix (patch) - fixes an issue
  • ✨ New feature (minor) - adds functionality
  • 💥 Breaking change (major) - changes existing behavior
  • 📝 Documentation - updates docs only
  • 🔧 Tooling/Config - CI, build, dependencies
  • ♻️ Refactor - code improvement, no behavior change
  • ⚡ Performance - improves performance

✅ Changes

update import in tests to use mjs in place of ts files.

⚠️ Breaking Changes

  • Yes (describe below and include migration guide in changeset)
  • No

📝 Checklist

  • Code follows project style (npm run lint passes)
  • Tests pass locally (npm test)
  • Commits follow conventional commits
  • Changeset added (if needed)
  • Documentation updated (if needed)
  • Types updated (if API changed)

@vikkrantxx7 vikkrantxx7 merged commit 26c8cb9 into main Nov 10, 2025
1 check passed
@sonarqubecloud
Copy link

@vikkrantxx7 vikkrantxx7 deleted the bugfix/fix-ts-file-import-in-tests branch November 10, 2025 17:09
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.

1 participant