Mezcla de ingles y español

[iheredia]

Hay dos lugares donde creo que la mezcla de español e ingles resalta y puede llegar a confundir:

• Dentro del código mismo. Ej, el grueso del codigo está en ingles pero los nombres de los metodos de la clase DataCleaner están en español. Esto impacta en el siguiente punto:
• La configuración de las reglas a aplicar tienen mezcla de español e ingles. Ej: {"remover_columnas": [{"field": "col1"} ]}

Atacar el primer punto no deberia ser complejo, pero si queremos atacar ambos y tener tanto código como config de reglas todo en ingles entonces se puede meter algun helper para tener backwards compatibility

[abenassi]

@iheredia creo que:

1. Podríamos taggear una última versión del proyecto antes de hacer esta traducción, y subir una versión minor, indicando la traducción al inglés y que esto rompe la compatibilidad hacia atrás. Si actualizás versión, tenés que corregir los viejos scripts.

2. Si se encara el refactor primero, es bastante fácil asegurar backwards de nombres con esto: cuando bindeas la función en el objeto principal lo hacés dos veces, una con el nombre en español y otra con el nombre en inglés pero apuntando a la misma función.

´´´
class DataJson(dict):
"""Objeto que representa un catálogo de activos de datos."""

# Variables por default
CATALOG_FIELDS_PATH = os.path.join(ABSOLUTE_PROJECT_DIR, "fields")

def __init__(self, catalog=None, schema_filename=None, schema_dir=None,
             default_values=None, catalog_format=None):

(.............)
push_dataset_to_ckan = federation.push_dataset_to_ckan
empujar_dataset_a_ckan = federation.push_dataset_to_ckan
harvest_dataset_to_ckan = federation.harvest_dataset_to_ckan
cosechar_dataset_a_ckan = federation.harvest_dataset_to_ckan
restore_dataset_to_ckan = federation.restore_dataset_to_ckan
restaurar_dataset_a_ckan = federation.restore_dataset_to_ckan

[iheredia]

Me gusta la opción 2 asi no rompemos compatibilidad hacia atras.
+1 a encarar esto despues del refactor