Skip to content

Backport PR 3857 and 4550 - #5277

Open
ytmimi wants to merge 4 commits into
rust-lang:mainfrom
ytmimi:backport_pr_3857_4550
Open

ytmimi wants to merge 4 commits into
rust-lang:mainfrom
ytmimi:backport_pr_3857_4550

Conversation

@ytmimi

@ytmimi ytmimi commented Mar 28, 2022

Copy link
Copy Markdown
Contributor

Back ported #3857 and #4550

Closes #5017
Closes #4515

topecongiro and others added 4 commits July 19, 2022 21:23
* Fix rust-lang#4502 issue - assignment indentation after one line comment

* Enhancement by using rewrite_assign_rhs_with_comments()

* Changes per comments received
Issue 5017 was fixed by backporting PR 3857 and PR 4550.
Closes 4515

After backporting 3857 and 4550 we were able to resolve 4515.
@ytmimi
ytmimi force-pushed the backport_pr_3857_4550 branch from 78b3a9e to b5225b7 Compare July 20, 2022 01:31
@ytmimi ytmimi added the P-low Low priority label Jul 27, 2022
@jieyouxu jieyouxu added the needs-triage This issue or PR needs triaging to determine its status. Remove label once sufficiently triaged. label Oct 10, 2025
@jieyouxu jieyouxu added S-waiting-on-review Status: awaiting review from the assignee but also interested parties. and removed pr-not-reviewed labels Feb 23, 2026
@rustbot

rustbot commented Mar 13, 2026

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (possibly #6823) made this pull request unmergeable. Please resolve the merge conflicts.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-triage This issue or PR needs triaging to determine its status. Remove label once sufficiently triaged. P-low Low priority S-waiting-on-review Status: awaiting review from the assignee but also interested parties.

Projects

None yet

5 participants