Primary key

[uptrace]

Primary key

[vmihailenco]

Should be fixed by 5176653

Looking at the code it should warn about the usage of pk as a column name as it is also an option

It is unrelated to this issue, but we may indeed improve the check in case of bun:"pk,pk.

[isgj]

Maybe by adding another pk option

type User {
    ID int64 `bun:"pk,pk,autoincrement"`
}

The first pk to tell that field ID is backed by pk column in db, the second to tell it's a primary key. Looking at the code it should warn about the usage of pk as a column name as it is also an option

if tag.Name != sqlName && isKnownFieldOption(tag.Name) {
	internal.Warn.Printf(
		"%s.%s tag name %q is also an option name; is it a mistake?",
		t.TypeName, f.Name, tag.Name,
	)
}

[sloan-dog]

Is the bun:"option1,option2,option3" format documented anywhere? I could not find a comprehensive list of struct tags and their components in current bun docs. Know it is a new -ish project, but derived from another. It was not apparent to me how to canonically indicate a field is a primary key in bun.

[vmihailenco]

Looks like that info was lost during migration :( I will restore it, but meanwhile you can check https://pg.uptrace.dev/models/#struct-tags

[vmihailenco]

Docs were updated - https://bun.uptrace.dev/guide/models.html#struct-tags