Skip to content

Conversation

Centri3
Copy link
Member

@Centri3 Centri3 commented Aug 2, 2023

Fixes #11285

Still an issue with the "from" side, i.e., I8::from(1) as I64 shows as i8 to I64, but this should be ok. Not possible to reliably fix currently anyway.

changelog: [cast_lossless]: Suggest type alias instead of the aliased type

@rustbot
Copy link
Collaborator

rustbot commented Aug 2, 2023

r? @llogiq

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 2, 2023
@bors
Copy link
Contributor

bors commented Aug 11, 2023

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

@llogiq
Copy link
Contributor

llogiq commented Oct 23, 2023

This one also looks merge-worthy after a rebase. @bors delegate+

@bors
Copy link
Contributor

bors commented Oct 23, 2023

✌️ @Centri3, you can now approve this pull request!

If @llogiq told you to "r=me" after making some further change, please make that change, then do @bors r=@llogiq

@llogiq
Copy link
Contributor

llogiq commented Feb 29, 2024

Any updates on this?

@Centri3
Copy link
Member Author

Centri3 commented Feb 29, 2024

I'll add a note about lazy_type_alias then rebase.

@Centri3
Copy link
Member Author

Centri3 commented Mar 12, 2024

@bors r=llogiq

@bors
Copy link
Contributor

bors commented Mar 12, 2024

📌 Commit 244d7da has been approved by llogiq

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Mar 12, 2024

⌛ Testing commit 244d7da with merge 92ca7c9...

@bors
Copy link
Contributor

bors commented Mar 12, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing 92ca7c9 to master...

@bors bors merged commit 92ca7c9 into rust-lang:master Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lossy type aliases in cast_lossless lint
4 participants