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

Rename folding/visiting traits #767

Merged
merged 6 commits into from
Jun 21, 2022
Merged

Rename folding/visiting traits #767

merged 6 commits into from
Jun 21, 2022

Conversation

eggyal
Copy link
Contributor

@eggyal eggyal commented Jun 19, 2022

Align the names of chalk's folding/visiting traits with those in rustc (especially once rust-lang/rust#98206) is merged.

r? @jackh726

@eggyal
Copy link
Contributor Author

eggyal commented Jun 19, 2022

Apologies for the force-push there. I tried to reorder the commits but it completely messed up the branch; didn't notice before pushing.

Failing test is due to broken doc links but this commit will update those docs? Test should probably be against revised documentation, no?

@detrumi
Copy link
Member

detrumi commented Jun 20, 2022

Failing test is due to broken doc links but this commit will update those docs? Test should probably be against revised documentation, no?

This was just fixed, so just ignore it in this PR.

@jackh726
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

📌 Commit 23d39c9 has been approved by jackh726

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

⌛ Testing commit 23d39c9 with merge ebdb87c...

@bors
Copy link
Collaborator

bors commented Jun 21, 2022

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

@bors bors merged commit ebdb87c into rust-lang:master Jun 21, 2022
@bors bors mentioned this pull request Jun 21, 2022
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