search

goraft / raft

UNMAINTAINED: A Go implementation of the Raft distributed consensus protocol.
MIT License
2.43k stars 480 forks source link

chore(*): gofmt everything #171

Closed philips closed 10 years ago

xiang90 commented 10 years ago

@philips I am cleaning all static data into a data package. So I am not going to merge this now.

philips commented 10 years ago

@xiangli-cmu That is fine, I just ran it in my editor while reviewing code. Just do it in your PR.

benbjohnson commented 10 years ago

@xiangli-cmu What static data?

xiang90 commented 10 years ago

@benbjohnson Snapshot, Request and Reply.

benbjohnson commented 10 years ago

@xiangli-cmu Why are you moving those to a separate package?

xiang90 commented 10 years ago

@benbjohnson Personally, I think there are too many files in the first level folder. And some of the files do no depend on each other. So it is cleaner to make them into separate pkgs.

benbjohnson commented 10 years ago

@xiangli-cmu They're all functionality related to the same thing. go-raft is meant to be a relatively small, self contained library. Splitting off into additional packages is overkill.