Suggestion: Use `import type { Kysely }` for migration template

acro5piano / kysely-migration-cli

A thin library to create a migration script using Kysely

MIT License

66 stars 3 forks source link

Closed m4rvr closed 7 months ago

m4rvr commented 9 months ago

Hey!

What do you think about using the explicit type import for Kysely in migration files?

Instead of import { Kysely } from 'kysely'

it would be import type { Kysely } from 'kysely'

acro5piano commented 9 months ago

@m4rvr That would be nice. Can you make a pull request? Thanks

acro5piano commented 7 months ago

Fixed on v0.3.0

m4rvr commented 7 months ago

Damn, sorry @acro5piano. Totally forgot about that 🙃 Thanks @marcosrjjunior for doing it! 🙏