Skip to content

Fail on warnings #61

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

Open
wants to merge 2 commits into
base: sources
Choose a base branch
from
Open

Fail on warnings #61

wants to merge 2 commits into from

Conversation

astrojuanlu
Copy link
Member

No description provided.

@astrojuanlu
Copy link
Member Author

Hay algunas imágenes a las que les falta el texto alternativo. Se podrían solucionar estos problemas sobre esta rama para ya nunca más tener warnings.

@manugarri
Copy link
Contributor

cojonudo, por mi dale. Por cierto, sabemos por que hay conmflictos con travis?

@astrojuanlu
Copy link
Member Author

Conflictos? No, lo que da es un fail porque solo he cambiado la configuración de pelican, los textos de las imágenes hay que añadirlos

@kikocorreoso kikocorreoso self-requested a review October 8, 2018 10:35
@kikocorreoso
Copy link
Member

Mergea esto si todavía aplica. Por mi +1000 a no ver warnings de esos.

@manugarri
Copy link
Contributor

@Pybonacci/administradores mergeamos? Creo que estamos todos de acuerdo.

@astrojuanlu
Copy link
Member Author

No podemos mergear, ¡está fallando!

@manugarri
Copy link
Contributor

Cierto, es posible que al quitar esos flags pelican simplemente falla si existen? En ese caso es incluso peor el remedio que la enfermedad, no?

@astrojuanlu
Copy link
Member Author

O sea, el arreglo es arreglar las imágenes realmente :)

@manugarri
Copy link
Contributor

eso parece :/, de todas formas no son muchas.

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.

3 participants