Feature/skip blocks #39
Conversation
coco.py
Outdated
|
|
||
| if helpers.is_course_url(url): | ||
| with Client(headless=headless) as client: | ||
| lst_errors_url = [] |
There was a problem hiding this comment.
El nombre lst_errors_url no es muy descriptivo. ¿Podrías considerar cambiarlo a list_of_errors o errors_list?
coco.py
Outdated
| except VideoError: | ||
| tprint("✗ Unable to fetch the video details.") | ||
| message = f"[SECTION] {section_title} [VIDEO] {video_url}" | ||
| message = f"[SECTION] {section_title} [VIDEO] {video_url.url}" |
There was a problem hiding this comment.
La variable video_url es de tipo String, por lo que no tiene la propiedad url. Debería ser simplemente {video_url}
coco.py
Outdated
|
|
||
| if len(lst_errors_url) > 0: | ||
| tprint("[bold red]URLs with ERROR[/bold red]") | ||
| for leu in lst_errors_url: |
There was a problem hiding this comment.
La variable leu podría cambiarse a solo error para ser más descriptivo.
|
Hola, @hopkeinst. Gracias por tomarte tu tiempo en contribuir con este proyecto. Solo tengo un par de observaciones no tan relevantes, pero que ayudarían a mejorar la claridad y legibilidad del código:
Gracias de nuevo por tu trabajo en este proyecto. Si tienes alguna pregunta, sugerencia o mejora puedes escribirme por Discord o por este medio. Saludos, |
Bugs: \n 1. Change video_url.url to video_url (this is string no object) \n 2. Rename variables for easy understanding
|
|
Hola @ivansaul Listo Primero que todo, me disculpo por lo del objeto en lugar del string (video_url -> video_url.url) se me pasó por andar trabajando en otra rama (no me dió tiempo para probar y solo hice el cambio y revisé el código por encima y que estuviera todo bien). También los cambios en los nombres de las variables. Probado y funcional 🆗 |
2 participants
#38
#38 (comment)
Se corre pre-commit y hacen cambios mencionados por @ivansaul en #38 (comment)