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

Corregidos links de ficheros de datos #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alexandregz
Copy link

Los .txt existentes en assets están con rutas que han sido modificadas, corregidas.

Basicamente ha sido cambiar http://www.infoelectoral.mir.es/infoelectoral/ por https://infoelectoral.interior.gob.es/estaticos/, el resto del path y los ficheros no han cambiado.

@JaimeObregon
Copy link
Owner

Muchas gracias, @alexandregz.

He echado un vistazo a la «pull request» que propones y aparentemente incorpora muchas más modificaciones de las que expresas en tu mensaje, relacionada con ficheros rotos en el Ministerio del Interior.

Además, mi código original está en inglés con comentarios en castellano —pues se trata de procesos electorales españoles— pero la «pull request» recibida tiene código y comentarios en gallego. Si la incorporo («merge»), la coherencia y la calidad del programa se verían afectadas.

Si deseas que la mezcle, por favor, reenvía una nueva «pull request» con los cambios mínimos necesarios para corregir los enlaces rotos al Ministerio solamente, en el mismo idioma que el código original.

En todo caso, la dejo abierta por si alguien quiere ayudar normalizando los estilos o de otra manera.

Gracias de nuevo.

@alexandregz
Copy link
Author

Hola @JaimeObregon,

me he equivocado y he añadido commits de un side project basado en tu código que estoy realizando.

Los voy a revertir para dejar solamente el commit en cuestión, el primero de los anotados.

@alexandregz
Copy link
Author

Done. En todo caso, si prefieres hago un commit limpio de nuevo. Saludos y gracias por el trabajo realizado, muy útil para la comunidad :-)

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

Successfully merging this pull request may close these issues.

2 participants