Skip to content

Commit 490749a

Browse files
Copilotgonzalocasas
andcommitted
Remove JsonMessageCodec from top-level exports
- Removed JsonMessageCodec from compas_eve.__init__ exports - Moved import back to lazy import in Transport.__init__ to avoid top-level dependency - JsonMessageCodec should be imported as `from compas_eve.codecs import JsonMessageCodec` - This keeps the API cleaner and more organized Co-authored-by: gonzalocasas <[email protected]>
1 parent ba9fcab commit 490749a

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/compas_eve/__init__.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
Transport
2222
InMemoryTransport
2323
MessageCodec
24-
JsonMessageCodec
2524
get_default_transport
2625
set_default_transport
2726
@@ -49,7 +48,7 @@
4948
get_default_transport,
5049
set_default_transport,
5150
)
52-
from .codecs import MessageCodec, JsonMessageCodec
51+
from .codecs import MessageCodec
5352
from .memory import InMemoryTransport
5453

5554
HERE = os.path.dirname(__file__)
@@ -66,7 +65,6 @@
6665
"Topic",
6766
"Transport",
6867
"MessageCodec",
69-
"JsonMessageCodec",
7068
"get_default_transport",
7169
"set_default_transport",
7270
"InMemoryTransport",

src/compas_eve/core.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
from compas_eve.codecs import JsonMessageCodec
2-
31
DEFAULT_TRANSPORT = None
42

53

@@ -41,7 +39,11 @@ class Transport(object):
4139
def __init__(self, codec=None, *args, **kwargs):
4240
super(Transport, self).__init__(*args, **kwargs)
4341
self._id_counter = 0
44-
self.codec = codec or JsonMessageCodec()
42+
if codec is None:
43+
from compas_eve.codecs import JsonMessageCodec
44+
45+
codec = JsonMessageCodec()
46+
self.codec = codec
4547

4648
@property
4749
def id_counter(self):

0 commit comments

Comments
 (0)