-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] 476 lift pointers #557
base: master
Are you sure you want to change the base?
Conversation
return nil | ||
} | ||
|
||
func (m *Item) Clone() *Item { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported method Item.Clone should have comment or be unexported
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported method Item.Clone should have comment or be unexported
57f5b55
to
df0cc9c
Compare
df0cc9c
to
50a743a
Compare
/*Item item | ||
|
||
swagger:model item | ||
*/ | ||
type Item struct { | ||
__setValues bitset.BitSet | ||
__nulls bitset.BitSet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't use underscores in Go names; struct field __nulls should be _Nulls
09a37dd
to
29eebe4
Compare
4046616
to
9c57f4c
Compare
9c57f4c
to
883819d
Compare
Any updates? |
Starts implementing the items discussed in #476
For this to work we need a custom serializer for all types, not just the "special" ones. So most of the work is around generating a serializer, to a lesser extent it is about pointer/no pointer decisions at render time.