Skip to content

chore: port test files and config to TypeScript#12

Merged
darcyclarke merged 1 commit intovltpkg:mainfrom
vltbaudbot:feat/port-to-typescript
Feb 27, 2026
Merged

chore: port test files and config to TypeScript#12
darcyclarke merged 1 commit intovltpkg:mainfrom
vltbaudbot:feat/port-to-typescript

Conversation

@vltbaudbot
Copy link

This PR completes the TypeScript migration for the reproduce package by porting the remaining JavaScript files to TypeScript.

Changes Made

Test Files Conversion

  • test/run.jstest/run.ts: Added TypeScript types and proper error handling
  • test/test.jstest/test.ts: Added TypeScript types and type assertions

Configuration Updates

  • tsconfig.json: Updated include to cover test/*.ts files
  • package.json:
    • Updated test script to use tsx instead of plain node
    • Added tsx as dev dependency for running TypeScript files directly
  • Test runner: Modified to use npx tsx --test for TypeScript test execution

Cleanup

  • Removed legacy ESLint configuration files (.eslintrc.json, .eslintignore)
  • All JavaScript files are now converted to TypeScript

Testing

  • npm run build - TypeScript compilation succeeds
  • npm test - All tests pass with new TypeScript setup
  • ✅ ESLint passes on all TypeScript files

The package now has a complete TypeScript codebase with proper type safety throughout.

- Convert test/run.js → test/run.ts with TypeScript types
- Convert test/test.js → test/test.ts with TypeScript types
- Update tsconfig.json to include test/ directory
- Update package.json test script to use tsx for TypeScript tests
- Add tsx dev dependency for running TypeScript files directly
- Remove legacy ESLint config files (.eslintrc.json, .eslintignore)
- Update test runner to use npx tsx for TypeScript test execution
@darcyclarke darcyclarke merged commit e47e515 into vltpkg:main Feb 27, 2026
1 of 3 checks passed
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