Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reparado links equivocados en .overrides #1111

Merged
merged 2 commits into from Oct 31, 2020

Conversation

@aroquega
Copy link
Contributor

@aroquega aroquega commented Oct 26, 2020

Closes #1012

@aroquega
Copy link
Contributor Author

@aroquega aroquega commented Oct 26, 2020

@eamanu @iam-agf @cmaureir está es mi solución para los links, ya que son muy pocos los overrides, reviso todos, fuerza bruta. Y aparte acorte los links del reviewers-guide. ¿Qué les parece?

@aroquega aroquega changed the title fix missing links to .overrides' source code and improve reviewers-guide's links Reparado links equivocados en .overrides Oct 26, 2020
@iam-agf
Copy link
Contributor

@iam-agf iam-agf commented Oct 27, 2020

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?

@aroquega
Copy link
Contributor Author

@aroquega aroquega commented Oct 27, 2020

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'] %}

This comment has been minimized.

@mondeja

mondeja Oct 27, 2020
Contributor

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.

This comment has been minimized.

@aroquega

aroquega Oct 27, 2020
Author Contributor

Tienes razón, yo tampoco pude verlo

This comment has been minimized.

@mondeja

mondeja Oct 28, 2020
Contributor

Parece que sólo se encuentra ahí para indicar qué es el directorio .overrides. Yo lo eliminaría ya que no sirve para nada.

This comment has been minimized.

@eamanu

eamanu Oct 29, 2020
Contributor

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

This comment has been minimized.

@mondeja

mondeja Oct 29, 2020
Contributor

También se podría añadir un simple test que compruebe que la lista y los archivos a referenciar se encuentran sincronizados.

This comment has been minimized.

@aroquega

aroquega Oct 31, 2020
Author Contributor

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.

Copy link
Contributor

@mondeja mondeja left a comment

Por mí lo veo perfecto, excepto por un pequeño cambio

@cmaureir cmaureir merged commit e1dea3b into python:3.8 Oct 31, 2020
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
docs/readthedocs.org:python-docs-es Read the Docs build succeeded!
Details
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Oct 31, 2020

Dejamos lo del test pendiente.
Muchas gracias @aroquega 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

5 participants
You can’t perform that action at this time.