Skip to content
This repository was archived by the owner on Nov 21, 2018. It is now read-only.

Fixed the targets of analysis manifest entries. #153

Merged
merged 1 commit into from
Feb 4, 2017
Merged

Fixed the targets of analysis manifest entries. #153

merged 1 commit into from
Feb 4, 2017

Conversation

Arnavion
Copy link
Contributor

@Arnavion Arnavion commented Feb 4, 2017

Before this change, all entries would use the last visited target from the
previous loop, thus would not match their parent target section.


Ref #150 (comment)

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @brson (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

Before this change, all entries would use the last visited target from the
previous loop, thus would not match their parent target section.
@brson brson merged commit 154d0b9 into rust-lang-deprecated:master Feb 4, 2017
@brson
Copy link
Contributor

brson commented Feb 4, 2017

Thanks! cc @nrc

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants