site: 'timestamp' instead of 'text' on migration example

[endel]

Small update on the SQLite migration example from the website: https://kysely.dev/docs/migrations#sqlite-migration-example

Replace this:

    .addColumn('created_at', 'text', (col) =>
      col.defaultTo(sql`CURRENT_TIMESTAMP`).notNull()
    )

With this:

    .addColumn('created_at', 'timestamp', (col) =>
      col.defaultTo(sql`CURRENT_TIMESTAMP`).notNull()
    )

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
kysely	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 4, 2023 10:41pm

[igalklebanov]

Hey 👋

Thank you! 💪

I'm not quite sure this should be encouraged:

a. there is no timestamp data type in sqlite.
b. timestamp has affinity type numeric (not documented), and numeric can store any data type (source). so having text there instead should be safer?

[endel]

Wow, thank you for the explanation and for sharing the article 🤯 Closing...