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

Los warnings de docutils deben estar apagados en producción #316

Open
pabloz opened this issue Aug 19, 2015 · 4 comments
Open

Los warnings de docutils deben estar apagados en producción #316

pabloz opened this issue Aug 19, 2015 · 4 comments

Comments

@pabloz
Copy link

pabloz commented Aug 19, 2015

En mi opinión, mensajes como este en http://python.org.ar/nosotros/:

<<<<<< Updated upstream
System Message: WARNING/2 (<string> line 2)

Title underline too short.

<<<<<<< Updated upstream
=======

deberían ser considerados problemas graves en producción y su origen arreglado de inmediato; cualquier cambio que genere este tipo de resultados (tanto si involucra contenidos, plantillas, lógica de la aplicación, herramientas o la plataforma entera) debería ser revertido ni bien es detectado y hasta que el problema de fondo quede solucionado. Ningún agregado debería justificar, en mi opinión, regresiones de este tipo.

Independientemente de lo anterior, los warnings de docutils deberían silenciarse en producción, e idealmente ser volcados en un log.

@angvp
Copy link
Contributor

angvp commented May 6, 2016

@gilgamezh

@gilgamezh
Copy link
Contributor

@angvp lo que yo entiendo es que eso no es un log de docutils. Que es un mensaje por un conflicto de git.

No se cómo se hace para que waliki no muestre eso.

@angvp
Copy link
Contributor

angvp commented May 7, 2016

@mgaitan Alguna idea?

@mgaitan
Copy link
Contributor

mgaitan commented May 9, 2016

hola muchachos. efectivamente, este caso particular es una mezcla de cosas: un rst que se guardó con conflictos y despues rst se confunde con la sintaxis sucia que queda y se queja.

Por qué un archivo se pudo guardar con conflictos es porque hubo una edición simultanea de la pagina que no se pudo automezclar. En ese caso waliki commitea el archivo "as is" (con conflicto) pero inmediatamente redirige al usuario al editor (ver acá), para que arregle y genere una nueva versión.

Es polémico guardar una versión con conflictos. quizas la actual podria ser una politica configurable y tener otra más simple "ultima edición gana" y chau.

Respecto al nivel de warnings, creo que bastaria con setear el nivel de report_level en settings WALIKI_MARKUPS_SETTINGS

No hace falta redefinir todo el dict, se extiende si la clave no está. entonces, algo asi

WALIKI_MARKUPS_SETTINGS = {'reStructuredText': {
    'settings_overrides': {
        'report_level': 5,
    }
}

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

No branches or pull requests

4 participants