box / box/box-python-sdk

Consider using wrapt with decorators

Abierto
#168 0 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement question
Lenguaje dominante
Python
Estrellas
459
Forks
223
Merge medio
8 h 57 min
PR fusionados (30 d)
13

Descripción

Now that we're using `@api_call` with all of our methods, they no longer can be introspected properly. For example, see http://box-python-sdk.readthedocs.io/en/latest/boxsdk.object.html#boxsdk.object.base_object.BaseObject.get .

We could potentially use wrapt (https://pypi.python.org/pypi/wrapt) to fix this. See https://github.com/GrahamDumpleton/wrapt/tree/develop/blog for more info.

Though this would potentially slow down the SDK code, so it might or might not be worth it. Perhaps it can be enabled when generating docs or using the terminal, but disabled otherwise.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Start at the @api_call decorator and compare the documented BaseObject.get behavior with the introspection issue described in the linked documentation. Read the referenced wrapt material, then determine whether preserving introspection is worth the potential SDK performance cost and whether behavior should differ for documentation or terminal use.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
developer-experience, documentation
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.