ruby / ruby/error_highlight

JRuby support

Offen
#72 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Ruby
Sterne
155
Forks
37
Ø Merge
4 T. 2 Std.
Gemergte PRs (30 T.)
1

Beschreibung

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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie mit den vier CRuby-spezifischen Verwendungen in lib/error_highlight/base.rb, insbesondere den Aufrufen von RubyVM::AbstractSyntaxTree und dem Prism::Node-Zweig. Untersuchen Sie, ob Prism die benötigten AST-Informationen zum Backtrace bereitstellen kann oder ob JRuby-spezifische Methoden erforderlich sind, und überprüfen Sie anschließend, ob die Funktionen zur Ausnahmehervorhebung des Gems unter JRuby funktionieren.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
ruby
Bereich
devtools
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.