fix: code style according PEP8 and endpoints naming #25
Conversation
src/test_solution.py
Outdated
| }) | ||
| response = client.post('/members', json={"first_name": "Tommy", | ||
| "id": 3443, | ||
| "age": 23, |
There was a problem hiding this comment.
@hchocobar aquí veo que intentaste generar un poco de indentación pero no ha salido bien. Mi sugerencia:
response = client.post('/members', json={
"first_name": "Tommy",
"id": 3443,
"age": 23,
"lucky_numbers": [34,65,23,4,6]
})
There was a problem hiding this comment.
O como la línea 37, que ha quedado mejor.
There was a problem hiding this comment.
@marcogonzalo, el motivo es identación.
Esta es una manera de escribir código en Python según PEP8 (no es la única)
Esta es la documentación: https://peps.python.org/pep-0008/#code-lay-out
There was a problem hiding this comment.
Sí, pero no está bien alineada de todos modos. Fíjate que hay unas lineas mucho más adentro y no a la misma altura de los anteriores. En el bloque de la Línea 37 sí están alineados, todos, en este bloque no.
There was a problem hiding this comment.
Esa es la idea Marco, alinear a la apertura del curly bracket de json={
Pero no identar a la altrua de client.post(
Corrección en lineas 30 y 31 Se reemplazan tab x espacios, esto soluciona la identación.
Update test_solution.py
@alesanchezr
Buen día!