Skip to content
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

Bump JuliaSyntax to 0.4.8 #53119

Closed
wants to merge 2 commits into from
Closed

Bump JuliaSyntax to 0.4.8 #53119

wants to merge 2 commits into from

Conversation

LilithHafner
Copy link
Member

@LilithHafner LilithHafner commented Jan 30, 2024

Using Mod as ModName (#52784) depends on the corresponding PR to JuliaSyntax
(JuliaLang/JuliaSyntax.jl#406) which I made against the commit to JuliaSytnax main.

This PR separates bumping JuliaSyntax from 0.4.7 to 0.4.8 so that #52784 is easier to review.

c.f. #52136, which bumped to 0.4.7

@vtjnash
Copy link
Sponsor Member

vtjnash commented Jan 30, 2024

checksums seem to be wrong. try regenerating those?

@LilithHafner
Copy link
Member Author

Some of the test failures were testing that we gave wrong line numbers and now we give better numbers. Some of the test failures look owrth looking into, though. For example,

@test Meta.lower(Main, :(a.[1])) == Expr(:error, "invalid syntax \"a.[1]\"")

@c42f

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.

None yet

2 participants