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
Remove submodules #1835
Remove submodules #1835
Conversation
Pull Request Checklist
|
Jenkins BuildsClick to see older builds (30)
|
0718ac2
to
29f6b86
Compare
29f6b86
to
6068ea1
Compare
@@ -58,7 +58,7 @@ type Contact struct { | |||
SystemTags []string `json:"systemTags"` | |||
|
|||
DeviceInfo []ContactDeviceInfo `json:"deviceInfo"` | |||
TributeToTalk string `json:"tributeToTalk,omitEmpty"` | |||
TributeToTalk string `json:"tributeToTalk,omitempty"` |
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.
Potentially breaking change but tests pass. cc @cammellos
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.
This is ok, I believe, tributeToTalk is not used
@@ -93,7 +93,7 @@ type Message struct { | |||
|
|||
// Replace indicates that this is a replacement of a message | |||
// that has been updated | |||
Replace string `json:"replace,omitEmpty"` | |||
Replace string `json:"replace,omitempty"` |
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.
Potentially breaking change cc @cammellos
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.
This is also likely ok
LocalChatID string `json:"localChatId"` | ||
Clock uint64 `json:"clock"` | ||
Replace string `json:"replace,omitEmpty"` | ||
Replace string `json:"replace"` |
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.
A unit test was failing so I removed it. omitEmpty
did not work, it is omitempty
. cc @cammellos
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.
good to know, I obviously sprinkled the codebase with the wrong version :)
Messages []*Message `json:"messages,omitEmpty"` | ||
Contacts []*Contact `json:"contacts,omitEmpty"` | ||
Installations []*multidevice.Installation `json:"installations,omitEmpty"` | ||
Chats []*Chat `json:"chats,omitempty"` |
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.
Potentially breaking change cc @cammellos.
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.
This is also fine
Messages []*Message `json:"messages,omitEmpty"` | ||
Contacts []*Contact `json:"contacts,omitEmpty"` | ||
Installations []*multidevice.Installation `json:"installations,omitEmpty"` | ||
Chats []*Chat `json:"chats,omitempty"` |
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.
This is also fine
At the current stage, submodules, especially for
protocol
andeth-node
packages, bring more troubles than advantages.The biggest problem is that we use
replace
s in the maingo.mod
which makes status-go act differently depending whether it is used as a main program vs a library in other projects.