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.

Sign up for GitHub

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

Jump to bottom

REF: fix auto-import in introduce constant refactoring #5379

Merged
merged 1 commit into from
May 17, 2020
Merged

REF: fix auto-import in introduce constant refactoring #5379

merged 1 commit into from
May 17, 2020

Conversation

Kobzol
Copy link
Member

@Kobzol Kobzol commented May 14, 2020

Fixes: #5377

@Undin Undin added the fix Pull requests that fix some bug(s) label May 15, 2020
@Undin Undin added this to the v122 milestone May 15, 2020
@vlad20012
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request May 17, 2020
@bors @Kobzol
Merge #5379
bfc03cc 
5379: REF: fix auto-import in introduce constant refactoring r=vlad20012 a=Kobzol

Fixes: #5377

Co-authored-by: Jakub Beránek <berykubik@gmail.com>
@bors
Copy link
Contributor

bors bot commented May 17, 2020

Build failed:

@vlad20012
Copy link
Member

bors retry

@bors
Copy link
Contributor

bors bot commented May 17, 2020

Build succeeded:

@bors bors bot merged commit 8f4962a into intellij-rust:master May 17, 2020
@Kobzol Kobzol deleted the introduce-constant-import-fix branch May 17, 2020 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@vlad20012 vlad20012

Labels
fix Pull requests that fix some bug(s)
Projects
None yet
Milestone
v122
Development

Successfully merging this pull request may close these issues.

"Introduce Constant" inserts a needless "use crate" when extracting to a file scope
3 participants
@Kobzol @vlad20012 @Undin