rel.String should be NVARCHAR

[billgraziano]

Disclaimer: I'm just reviewing the code while deciding whether to use or not. I haven't used this for real yet.

I believe a GO string (Unicode) best matches to NVARCHAR instead of VARCHAR. That will support the full Unicode set. At least that's what I've done in my code and it's worked well.

mssql/mssql.go

Lines 137 to 142 in e7d7d12

	case rel.String:
	typ = "VARCHAR"
	m = column.Limit
	if m == 0 {
	m = 255
	}

[Fs02]

Thank you again for this great feedback, I've created a pr to address this issue.
please take a look if you have time 😄

FYI: This column mapping is only used for migration functionality and it's also possible to explicitly define corresponding MSSQL type using the following function:

func (t *Table) Column(name string, typ ColumnType, options ...ColumnOption)

this issue gave me an idea to create extension type on this driver, to better cover all available types in MSSQL, thanks