Skip to content

save-analysis: better 'parent' info #36487

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Sep 19, 2016
Merged

Conversation

nrc
Copy link
Member

@nrc nrc commented Sep 15, 2016

In particular, this fixes some bugs displaying doc URLs for method calls.

@nrc
Copy link
Member Author

nrc commented Sep 16, 2016

r? @eddyb

Copy link
Member

@eddyb eddyb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me modulo the submodule accident.

@@ -1 +1 @@
Subproject commit d4f6a19c55a03e3f9f6fb7377911b37ed807eb6c
Subproject commit 49d64cae0699ed9d9ed84810d737a26b0b519da8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Accidental submodule change? I'd advise against git commit -a in the future.

@nrc nrc force-pushed the save-doc-urls branch 3 times, most recently from a7c5d5d to eece652 Compare September 18, 2016 19:08
In particular, this fixes some bugs displaying doc URLs for method calls.
@nrc
Copy link
Member Author

nrc commented Sep 18, 2016

@bors: r=@eddyb

@bors
Copy link
Collaborator

bors commented Sep 18, 2016

📌 Commit 48e69e0 has been approved by @eddyb

@bors
Copy link
Collaborator

bors commented Sep 18, 2016

⌛ Testing commit 48e69e0 with merge 55bf6a4...

bors added a commit that referenced this pull request Sep 18, 2016
save-analysis: better 'parent' info

In particular, this fixes some bugs displaying doc URLs for method calls.
@bors bors merged commit 48e69e0 into rust-lang:master Sep 19, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants