mumuki-laboratory icon indicating copy to clipboard operation
mumuki-laboratory copied to clipboard

Add gutter icon in code mirror to display line error

Open aguspina opened this issue 7 years ago • 4 comments

We need to add the error line number in the mumukit output, so as to give laboratory the necessary info to add a gutter icon in the corresponding line.

aguspina avatar Nov 27 '17 14:11 aguspina

Hay que cambiar mumukit para que agregue a la salida inline_messages:

{
  
   exit: ...
   result: ...
   test_results: ....
   type: ....
   inline_messages: [
      { type: 'error',  message: 'ups aca esto no compila', line: 2, column: 4 },
      { type: 'warning',  }
   ]
}

Hay que revisar Bridge (tenemos que ver si es necesario cambiar algo).

Hay que cambiar labo: para que trabaje con esta salida.

Hay que cambiar algún runner (por ejemplo, el de java) para que reporte esta información.

  • puede que necesitemos otro hook
  • puede que lo agregemos en el feedback hook
  • puede que introduzcamos la idea de un observador del request

flbulgarelli avatar Nov 29 '17 18:11 flbulgarelli

@flbulgarelli, does this issue still make sense?

fedescarpa avatar Aug 27 '19 14:08 fedescarpa

Sip.

flbulgarelli avatar Aug 27 '19 15:08 flbulgarelli

Últimamente yo trabajo mucho más con bloques pero cuando trabajas con texto es muy difícil encontrar errores con tan poca información contextual

flbulgarelli avatar Aug 27 '19 15:08 flbulgarelli