Rename the confusing object name TrxFile ?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- python
- Área
- developer-experience
Línea de trabajo
Comienza revisando el objeto TrxFile y la función concatenate, que el issue identifica como una superficie de API confusa. Determina con los maintainers el nombre de reemplazo previsto y, después, sigue el impacto de esa decisión de nomenclatura en todo el proyecto. Se considera terminado cuando el nuevo nombre y el alcance de la migración están acordados y se aplican de forma coherente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi trx team
TrxFile is a disturbing object since you expect or believe to have a filename, filepath, etc...
I propose to rename this object TrxContainer, TrxHolder, etc....
I am open to any other renaming but I strongly believe that TrxFile is very confusing.
concatenate function is a nice exemple. When you see TrxFile as an input of this function, I believed that I need to put a filename, however, in reality, you need to load it before calling the function.
Please, let me know what you think @frheault and @arokem
- Lenguaje dominante
- Python
- Estrellas
- 27
- Forks
- 20
- Merge medio
- 2 d 25 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de tee-ar-ex/trx-python
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
tee-ar-ex/trx-python#85 · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
tee-ar-ex/trx-python#77 · 1 comentario ·
-
Circular Dependency for DIPY ?Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
tee-ar-ex/trx-python#51 · 23 comentarios ·
-
Allow other types into `append`Abiertogood first issue
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
tee-ar-ex/trx-python#39 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
tee-ar-ex/trx-python#21 · 4 comentarios ·
Todos los issues de tee-ar-ex/trx-python
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
jbacalso24/mimry#291 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 Menos de una hora Aptitud para principiantes 82/100
eli-alvarado/us-watchlist-alerts#14 ·
Los mantenedores suelen responder en 1 día
-
from-review-extraction python severity:nit
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
LearningCircuit/local-deep-research#7462 ·
Los mantenedores suelen responder en 2 días
-
mermaid_extract.py silently discards single-line state-diagram notes; sequence notes are keptAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
cathrynlavery/diagram-design#441 ·
Los mantenedores suelen responder en 2 días
-
[Cleanup] Include the public header in work_split.cppPosiblemente ocupada @blozano-tt la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
tenstorrent/tt-metal#60850 ·
Los mantenedores suelen responder en 1 día