Added test for `StartupSettings` and documentation of use - in `README.md`

edwardcapriolo / gossip

A Mavenized Apache V2 gossip implementation for Java

Apache License 2.0

160 stars 54 forks source link

Added test for `StartupSettings` and documentation of use - in `README.md` #14

Closed irstevenson closed 8 years ago

irstevenson commented 8 years ago

Also tweaked the StartupSettings so that it now reads in an id for the node, rather than just resulting in a blank id
Moved some logging from stdout to log4j
Full maven test passing

Arguably this would result in a version change, as the settings file format has changed and now requires an id field. However we could make it optional, but it seems odd to have nodes with no ids - so if people update to the new version then they should do this.

edwardcapriolo commented 8 years ago

This makes sense. We can not run without ID anymore.