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

Remove unused Result assoc type from Fold trait #768

Merged
merged 1 commit into from
Jun 21, 2022

Conversation

eggyal
Copy link
Contributor

@eggyal eggyal commented Jun 19, 2022

@eggyal
Copy link
Contributor Author

eggyal commented Jun 19, 2022

Broken doc links look good from here, but evidently not working in CI.

@detrumi
Copy link
Member

detrumi commented Jun 20, 2022

Doc links are fixed now. No need to retry the build since the main build succeeded already.

@jackh726
Copy link
Member

@bors delegate+

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

✌️ @eggyal can now approve this pull request

@jackh726
Copy link
Member

After #767 lands, this will need a rebase. LGTM

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

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

@eggyal
Copy link
Contributor Author

eggyal commented Jun 21, 2022

Rebased.

@jackh726
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

📌 Commit a9d086c has been approved by jackh726

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

⌛ Testing commit a9d086c with merge 4991b7d...

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

☀️ Test successful - checks-actions
Approved by: jackh726
Pushing 4991b7d to master...

@bors bors merged commit 4991b7d into rust-lang:master Jun 21, 2022
@eggyal eggyal deleted the remove-fold-result-type branch January 10, 2023 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants