Reparado links equivocados en .overrides #1111
Conversation
|
Pues, por como lo veo (y entiendo), la fuerza bruta que mencionas termina siendo una búsqueda lineal después de todo, así que no le veo mucho problema, ¿No? |
|
Sí, termina siendo una búsqueda líneal en la compilación, después cuando se contruye, el html aparecerá con el url correcto. |
| Las siguientes líneas están para que se pueda redireccionar | ||
| al correctamente el código fuente en python-docs-es.readthedocs.io/ | ||
| #} | ||
| {% set overriden_pages = ['coc', 'CONTRIBUTING', 'faq', 'reviewers-guide','progress', 'README', 'translation-memory', 'upgrade-python-version'] %} |
There was a problem hiding this comment.
El archivo README creo que no se renderiza, al menos no soy capaz de acceder a él en la URL https://python-docs-es.readthedocs.io/es/3.8/README.html
¿Sabes en qué URL se encuentra? Si no, podría eliminarse de la lista.
There was a problem hiding this comment.
Tienes razón, yo tampoco pude verlo
There was a problem hiding this comment.
Parece que sólo se encuentra ahí para indicar qué es el directorio .overrides. Yo lo eliminaría ya que no sirve para nada.
There was a problem hiding this comment.
Esta lista la deberíamos poder armar automáticamente, porque si se quiere agregar un nuevo archivo o eliminarlo, deberíamos estar modificando esta línea. Aunque creo que se espera que no cambie demasiado
There was a problem hiding this comment.
También se podría añadir un simple test que compruebe que la lista y los archivos a referenciar se encuentran sincronizados.
There was a problem hiding this comment.
Un simple test, pero eso no tengo idea de cómo hacerlo. Creo que la lista se puede sacar dinámicamente con sólo sacar los .rst del .overrides en el mismo código donde se crea el banner de "ayúdanos a contribuir..." pero no sé qué tan eficiente sería eso.
|
Dejamos lo del test pendiente. |
Closes #1012