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.

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

Properly colorize multi-part suggestions in the same line #108627

Merged
merged 1 commit into from
Mar 5, 2023

Conversation

estebank
Copy link
Contributor

@estebank estebank commented Mar 1, 2023

Fix #108547.

@rustbot
Copy link
Collaborator

rustbot commented Mar 1, 2023

r? @Nilstrieb

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 1, 2023
@WaffleLapkin
Copy link
Member

r? @WaffleLapkin
@bors r+

@bors
Copy link
Collaborator

bors commented Mar 2, 2023

📌 Commit 3cea838 has been approved by WaffleLapkin

It is now in the queue for this repository.

@rustbot rustbot assigned WaffleLapkin and unassigned Noratrieb Mar 2, 2023
@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 2, 2023
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Mar 2, 2023
…WaffleLapkin

Properly colorize multi-part suggestions in the same line

Fix rust-lang#108547.
@matthiaskrgr
Copy link
Member

@bors r- failed in a rollup
maybe the colors are just a bit off..?
test output looks a bit sus
https://github.com/rust-lang-ci/rust/actions/runs/4317906822/jobs/7535564938#step:26:22033

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 2, 2023
@estebank
Copy link
Contributor Author

estebank commented Mar 2, 2023

@matthiaskrgr Oh no, it is because windows uses a different palette from linux and mac 🤦

I'll make it ignore windows for now.

@estebank estebank force-pushed the suggestion-hightlight branch from 3cea838 to 6fd1751 Compare March 4, 2023 00:30
@estebank
Copy link
Contributor Author

estebank commented Mar 4, 2023

@bors r=WaffleLapkin

@bors
Copy link
Collaborator

bors commented Mar 4, 2023

📌 Commit 6fd1751 has been approved by WaffleLapkin

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 4, 2023
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Mar 4, 2023
…WaffleLapkin

Properly colorize multi-part suggestions in the same line

Fix rust-lang#108547.
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 4, 2023
…iaskrgr

Rollup of 7 pull requests

Successful merges:

 - rust-lang#108627 (Properly colorize multi-part suggestions in the same line)
 - rust-lang#108632 (Omit unchanged options from config.toml in `configure.py`)
 - rust-lang#108715 (Remove unclosed_delims from parser)
 - rust-lang#108723 (rustdoc: function signature search with traits in `where` clause)
 - rust-lang#108724 (field is not used outside the crate)
 - rust-lang#108734 (rustdoc: Note in a type's layout/size if it is uninhabited)
 - rust-lang#108736 (Remove `allow(potential_query_instability)` from `ast_passes`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 60f54b1 into rust-lang:master Mar 5, 2023
@rustbot rustbot added this to the 1.69.0 milestone Mar 5, 2023
@estebank estebank deleted the suggestion-hightlight branch November 9, 2023 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

missing lifetime specifier error ui coloration misaligned
6 participants