search

allegro / marathon-consul

Integrates Marathon apps with Consul service discovery.
Apache License 2.0
191 stars 33 forks source link

Deogierize consul #195

Closed wendigo closed 7 years ago

wendigo commented 7 years ago

Adresses https://github.com/allegro/marathon-consul/issues/193

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.02%) to 90.608% when pulling 22ca1571a4f9c9ce40260d5a4574e7754f605ca3 on remove_ogier into bb2478ee75e585a6177403e80f526b1505802daa on master.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.02%) to 90.608% when pulling cb850c125a2ffad433521c919a4ce6ddfbff5d14 on remove_ogier into bb2478ee75e585a6177403e80f526b1505802daa on master.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.02%) to 90.731% when pulling 859ef8c32997f027089d3f71403c2070d2dc91d6 on remove_ogier into 329d53ad3c2b8d5036f5547c9f567767accba5d3 on master.

janisz commented 7 years ago

Since this change breaks backward compatibility, it should wait for version 1.3.0 or should be merged to develop.

wendigo commented 7 years ago

Sure thing @janisz

janisz commented 7 years ago

Closing due to missing backward compatibility.