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

Include import name in import shadowing error messages. #16598

Merged
merged 3 commits into from
Aug 30, 2014

Conversation

bkoropoff
Copy link
Contributor

This partially alleviates the confusing behavior in issue #16597

This partially alleviates the confusing behavior in issue rust-lang#16597
@Kimundi
Copy link
Member

Kimundi commented Aug 19, 2014

Hm, afaik there are tests you'd have to adjust as well... Did you run the full make check?

@bkoropoff
Copy link
Contributor Author

Yeah, oops. There was also a missing case, so I added that.

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Aug 29, 2014
bors added a commit that referenced this pull request Aug 30, 2014
This partially alleviates the confusing behavior in issue #16597
@bors bors closed this Aug 30, 2014
@bors bors merged commit 6bbec28 into rust-lang:master Aug 30, 2014
@bkoropoff bkoropoff deleted the import-shadow-name branch September 20, 2014 21:21
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 25, 2024
internal: Downgrade actions/checkout in release workflow
# 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.

4 participants