AllenNeuralDynamics / AllenNeuralDynamics/one-liner

Make `Client.version` and `Client.server_version` properties

Aperta
#38 1 commento 0 reazioni 1 assegnatario Rivendicata da @Poofjunior Vedi su GitHub
Lingua principale
Python
Stelle
3
Fork
0
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Current code in client.py

```python
def version(self):
"""Return client version."""
return local_version

def server_version(self):
"""Return the server version."""
return self.rpc_client.call("__router_server", "version")[1]
```

Should be

```python
@property
def version(self):
"""Return client version."""
return local_version

@property
def server_version(self):
"""Return the server version."""
return self.rpc_client.call("__router_server", "version")[1]
```

Or potentially `@cached_property`

This allows for the more idiomatic user experience of `client.version` rather than `client.version()`

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.