[BUG]: Outdated docs

[gzuuus]

What version of drizzle-orm are you using?

0.28.5

What version of drizzle-kit are you using?

0.19.13

Describe the Bug

Hey I was trying to work with composite primary keys in sqlite but the documentation is out of date and the syntax seems to be wrong as it says columns is not in type SQLiteColumn.
The object literal can only specify known properties, and 'columns' does not exist in the type 'SQLiteColumn<Required<{ name: string; dataType: ColumnDataType; columnType: string; data: unknown; driverParam: unknown; notNull: boolean; hasDefault: boolean; enumValues: string[] | undefined; tableName: string; }>, object>'.

This is the code I'm testing:

(table) => {
  return {
    pk: primaryKey({ columns: [table.bookId, table.authorId] }),
  };
}

Please fix the docs to represent the current way of doing composite primary keys.
I also noticed that the numeric type for sqlite is not listed in the docs, even though it can be used.

Expected behavior

No response

Environment & setup

No response

[realmikesolo]

Hello @gzuuus! Could you please show your schema? The issue might be caused by something else because docs are updated for composite primary key. I could create composite primary key with this code:

import { integer, primaryKey, sqliteTable } from 'drizzle-orm/sqlite-core';

export const tests = sqliteTable(
  'tests',
  {
    authorId: integer('author_id'),
    bookId: integer('book_id'),
  },
  (t) => ({
    pk: primaryKey({ columns: [t.bookId, t.authorId] }),
  }),
);

[gzuuus]

It seems to be fixed by updating the package, at the moment there is no error, however I have found several parts of the documentation to be out of date, as mentioned in the issue numeric type for sqlite is not listed in the docs, even though it can be used. Also the foreign key example in sqlite uses the deprecated method with the fallback https://orm.drizzle.team/docs/indexes-constraints#foreign-key

[realmikesolo]

@gzuuus thank you. Docs were fixed. I think there is no numeric in the docs, because it is not a datatype, numeric is an affinity according to SQLite docs

[gzuuus]

I see! In any case, it would be nice to have it documented in the documents so that people know they can use it.

[realmikesolo]

Agree with you! We will add info about numeric