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

Soft deprecate Sink #114384

Closed
wants to merge 3 commits into from
Closed

Soft deprecate Sink #114384

wants to merge 3 commits into from

Conversation

SUPERCILEX
Copy link
Contributor

@rustbot
Copy link
Collaborator

rustbot commented Aug 2, 2023

r? @Mark-Simulacrum

(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-libs Relevant to the library team, which will review and decide on the PR/issue. labels Aug 2, 2023
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rustbot
Copy link
Collaborator

rustbot commented Aug 2, 2023

Some changes occurred in src/tools/clippy

cc @rust-lang/clippy

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Aug 4, 2023

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

Signed-off-by: Alex Saveau <[email protected]>
Signed-off-by: Alex Saveau <[email protected]>
@rust-log-analyzer

This comment has been minimized.

@Mark-Simulacrum Mark-Simulacrum added the I-libs-api-nominated Nominated for discussion during a libs-api team meeting. label Aug 6, 2023
@Mark-Simulacrum
Copy link
Member

Nominating for libs-api to decide if we want to do this.

@Amanieu
Copy link
Member

Amanieu commented Aug 9, 2023

We discussed this in the libs-api meeting yesterday. We don't see any significant benefits to deprecating Sink. Instead it would be sufficient to just encourage the use of Sink when reading from the file is not required, and Empty if it is. The documentation for both should be updated to mention the other and under which circumstances it would be appropriate to use it.

@Amanieu Amanieu closed this Aug 9, 2023
@Amanieu Amanieu removed the I-libs-api-nominated Nominated for discussion during a libs-api team meeting. label Aug 9, 2023
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. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants