-
-
Notifications
You must be signed in to change notification settings - Fork 292
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
Upgrade codemirror with new parser #2980
Conversation
Co-Authored-By: Sergio A. Vargas <savargasqu+git@unal.edu.co>
Try this Pull Request!Open Julia and type: julia> import Pkg
julia> Pkg.activate(temp=true)
julia> Pkg.add(url="https://github.com/fonsp/Pluto.jl", rev="codemirror-savq")
julia> using Pluto |
Co-Authored-By: Sergio A. Vargas <savargasqu+git@unal.edu.co>
@savq mabye you could help with updates to lezer_template and scopestate_statefield? I think the rest is easy enough for me :) |
This PR got stuck because of the pattern matching code by dralletje. I gave it a try but this is way to hard to update, I'm declaring it unmaintainable! Since we barely use the pattern matching and scopestate system, I will look into removing it and rewriting functionality into simpler (less precise) tree traversal code. |
* Remove frontend AST pattern matching (WIP) * pkg bubble plugin without lezer template * uhmmm * remove log * write new scope analysis based on TreeCursor and remove old code * more precise function stuff * support for loop, fix weird iteration * Support try catch, fix console group, comments * Support module, quote, :, do * New API: Pluto.is_single_expression (#3134) * Update scopestate_statefield.js * Fix #3136 * Fix #3136 * Update scopestate_statefield.js * Fix bracket matching * Update block_matcher_plugin.js * Update scopestate_statefield.js * fix ts errors * Support scope in generator expressions
Ready! Merging soon :) |
@savq did incredible work on the Julia parser! JuliaPluto/lezer-julia#7
Fix #2943