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
ANN: Add quick fix to correct invalid cfg predicates #7165
ANN: Add quick fix to correct invalid cfg predicates #7165
Conversation
9cc7271
to
c5e7ed5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM except minor comments.
Also, the PR has merge conflicts. Could you rebase onto master, please?
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
588fb1b
to
f6cc7a2
Compare
Rebased & fixed comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! 👍
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/org/rust/ide/annotator/fixes/NameSuggestionFix.kt
Outdated
Show resolved
Hide resolved
And could you add |
f6cc7a2
to
f78a5e7
Compare
bors r=Undin |
Build succeeded: |
Improves #7102 by adding a quick fix to correct invalid cfg name.
Note that I couldn't just use
RenameFix
, because this fix has to work onRsPath
in the case ofcfg
predicates. And in this way the fix is general and it can be used in more places.changelog: Add quick fix to correct the name of an invalid cfg predicate.