Skip to content

Cargo update#1048

Merged
kngwyu merged 1 commit intoracer-rust:masterfrom
topecongiro:rustc-ap-syntax
Jun 9, 2019
Merged

Cargo update#1048
kngwyu merged 1 commit intoracer-rust:masterfrom
topecongiro:rustc-ap-syntax

Conversation

@topecongiro
Copy link
Contributor

Update rustc-ap-syntax to 491.0.0.

Note that I am just passing edition 2018 to syntax::with_globals. Ideally, we should pass the correct edition to it, but I find it to require quite a lot of code refactoring, and I basically gave up. Since the current implementation of racer does not seem to properly set the correct edition anyway, I am assuming that there will be no breaking change introduced with this PR.

Update rustc-ap-syntax to 491.0.0.
@topecongiro
Copy link
Contributor Author

cc rust-lang/rustfmt#3611.

@kngwyu
Copy link
Collaborator

kngwyu commented Jun 9, 2019

Thanks!

Since the current implementation of racer does not seem to properly set the correct edition anyway, I am assuming that there will be no breaking change introduced with this PR.

Thanks you for trying that.
The problem is calling cargo metadata can be slow but anyway we have to do do that

@kngwyu kngwyu merged commit 33cb382 into racer-rust:master Jun 9, 2019
@topecongiro
Copy link
Contributor Author

Could you make a point release so that we could update the dependencies of rls?

@topecongiro topecongiro deleted the rustc-ap-syntax branch June 9, 2019 23:21
@kngwyu
Copy link
Collaborator

kngwyu commented Jun 10, 2019

Sure. Now I'm doing some bug fixes and I'll release it by today.

@kngwyu
Copy link
Collaborator

kngwyu commented Jun 10, 2019

Released 2.1.23 now.

@topecongiro
Copy link
Contributor Author

@kngwyu Thank you!

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