Skip to content

Conversation

@Veykril
Copy link
Member

@Veykril Veykril commented Sep 22, 2022

No description provided.

@Veykril
Copy link
Member Author

Veykril commented Sep 22, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Sep 22, 2022

📌 Commit 729a9eb has been approved by Veykril

It is now in the queue for this repository.

bors added a commit that referenced this pull request Sep 22, 2022
Fix find_path using the wrong module for visibility calculations
@bors
Copy link
Contributor

bors commented Sep 22, 2022

⌛ Testing commit 729a9eb with merge 22901ee...

@Veykril
Copy link
Member Author

Veykril commented Sep 22, 2022

Tested that this fixes the regression that was found, but I can't reproduce it in a test. Couldn't figure out a minimization for it.

@bors
Copy link
Contributor

bors commented Sep 22, 2022

💔 Test failed - checks-actions

@Veykril
Copy link
Member Author

Veykril commented Sep 25, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Sep 25, 2022

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

@bors
Copy link
Contributor

bors commented Sep 25, 2022

📌 Commit 729a9eb has been approved by Veykril

It is now in the queue for this repository.

@Veykril
Copy link
Member Author

Veykril commented Sep 25, 2022

@bors r-

@Veykril
Copy link
Member Author

Veykril commented Sep 25, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Sep 25, 2022

📌 Commit 729a9eb has been approved by Veykril

It is now in the queue for this repository.

@Veykril
Copy link
Member Author

Veykril commented Sep 26, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Sep 26, 2022

💡 This pull request was already approved, no need to approve it again.

@bors
Copy link
Contributor

bors commented Sep 26, 2022

📌 Commit 729a9eb has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 27, 2022

⌛ Testing commit 729a9eb with merge 8c7e376...

@bors
Copy link
Contributor

bors commented Sep 27, 2022

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 8c7e376 to master...

@bors bors merged commit 8c7e376 into rust-lang:master Sep 27, 2022
@Veykril Veykril deleted the find-path branch September 27, 2022 09:43
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.

2 participants