-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Accept any Into<AnyNodeRef>
as Comments
arguments
#5205
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
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.
neat
@MichaReiser started a stack merge that includes this pull request via Graphite. |
Graphite rebased this pull request as part of a merge. |
d4de0d2
to
d6272b0
Compare
Graphite couldn't merge this PR because it was not satisfying all requirements. |
d6272b0
to
d90460d
Compare
PR Check ResultsEcosystem✅ ecosystem check detected no changes. BenchmarkLinux
Windows
|
Summary
This PR makes the
Comments
methods generic so that they accept anyInto<AnyNodeRef>
. This removes the need to call.into()
in the formatter code.Test Plan
cargo build