-
Notifications
You must be signed in to change notification settings - Fork 932
Closed
Description
These settings will result in an invalid config:
...
overrides:
- go_type: "encoding/json.RawMessage"
db_type: "json"
- db_type: "uuid"
null: true
go_type: "*github.com/google/uuid.UUID"
The reason is tha null
is a Keyword in YAML. However, this would be fine if I simply got an error using this invalid config. But I don't. instead the null
value above is silently ignored and will always default to false
.
I can fix the issue by quoting 'null'
like this:
...
overrides:
- go_type: "encoding/json.RawMessage"
db_type: "json"
- db_type: "uuid"
'null': true
go_type: "*github.com/google/uuid.UUID"
However, I think this setting name should be deprecated and something like nullable
being used instead, and printing some type of warning or similar
Metadata
Metadata
Assignees
Labels
No labels