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

Use getters to access Span fields #1899

Closed
wants to merge 1 commit into from

Conversation

petrochenkov
Copy link
Contributor

Required for merging rust-lang/rust#43968

r? @nrc

@nrc
Copy link
Member

nrc commented Aug 20, 2017

@petrochenkov
Copy link
Contributor Author

@nrc
rust-lang/rust#44028 was just merged.
Could you rebase https://github.com/rust-lang-nursery/rls/tree/rustfmt-span on master?

@nrc
Copy link
Member

nrc commented Aug 27, 2017

@petrochenkov rustfmt and rls branches are both rebased

@petrochenkov
Copy link
Contributor Author

Rebased.
rust-lang/rust#43968 was merged yesterday, so this PR can be merged directly now.

@nrc
Copy link
Member

nrc commented Aug 31, 2017

I merged this after a rebase. And it is getting pulled into Rust via the RLS via rust-lang/rust#44176

@nrc nrc closed this Aug 31, 2017
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