Skip to content

Conversation

@nsajko
Copy link
Member

@nsajko nsajko commented Sep 3, 2025

Type stability fix: makes the return value of the Expr constructor method infer as Expr.

Not sure if this could be fixed at a deeper level of the call stack instead.

Works around issue JuliaLang/julia#42372 in the case of this method.

Type stability fix: makes the return value of the `Expr` constructor
method infer as `Expr`.

Not sure if this could be fixed at a deeper level of the call stack
instead.

Works around issue JuliaLang/julia#42372 in the case of this method.
nsajko added a commit to nsajko/JuliaSyntax.jl that referenced this pull request Sep 3, 2025
@c42f
Copy link
Member

c42f commented Nov 20, 2025

Heya, we just moved JuliaSyntax into Base in JuliaLang/julia#59870 - if you want to re-propose this PR over there, I've created a branch https://github.com/JuliaLang/JuliaSyntax.jl/tree/pr-for-Base/592 to make this easier.

To make use of the branch, you can use the following steps:

  • Clone JuliaLang/julia
  • Add JuliaSyntax as a git remote
  • Check out pr-for-Base/592
  • Rebase on top of master

For example:

# git clone git@github.com:JuliaLang/julia julia_dir
# cd julia_dir

git remote add JuliaSyntax git@github.com:JuliaLang/JuliaSyntax.jl
git fetch JuliaSyntax
git checkout pr-for-Base/592
git rebase origin/master

Sorry this wasn't dealt with prior to the big move!

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

Successfully merging this pull request may close these issues.

2 participants