ruby / ruby/error_highlight

JRuby support

Abierto
#72 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Ruby
Estrellas
155
Forks
37
Merge medio
4 d 2 h
PR fusionados (30 d)
1

Descripción

This gem is unusable on anything but CRuby mostly because it uses the following CRuby-specific features:

lib/error_highlight/base.rb  (4 usages found)
    59           RubyVM::AbstractSyntaxTree.of(loc, keep_script_lines: true)
    70     when RubyVM::AbstractSyntaxTree::Node, Prism::Node
    93     node_id = RubyVM::AbstractSyntaxTree.node_id_for_backtrace_location(location)

We could fix this for JRuby in one of two ways:

  • Switch to the Prism gem where possible. I'm not sure if it can help us get a specific AST node from a backtrace element without reparsing, though. cc @kddnewton
  • Use JRuby-specific methods to access similar internal code structures.

It would be nice if JRuby users could benefit from the features of this gem.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con los cuatro usos específicos de CRuby enumerados en lib/error_highlight/base.rb, especialmente las llamadas a RubyVM::AbstractSyntaxTree y la rama de Prism::Node. Investiga si Prism puede proporcionar la información del AST necesaria del backtrace o si se requieren métodos específicos de JRuby; después, verifica que las funciones de resaltado de excepciones del gem funcionen en JRuby.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
ruby
Área
devtools
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.