Skip to content

Conversation

@alex-semenyuk
Copy link
Member

@alex-semenyuk alex-semenyuk commented Jul 28, 2022

Verified that it actully works on windows
changelog: none

@rust-highfive
Copy link

r? @dswij

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jul 28, 2022
Copy link
Member

@dswij dswij left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I couldn't find the reason we're ignoring this though, curious about this.

@xFrednet
Copy link
Contributor

xFrednet commented Jul 28, 2022

I'm guessing it had something to do with the Compiler flag? 🤔

@alex-semenyuk
Copy link
Member Author

Not sure why, but seems it was here #5086 Probably, @Areredify remember why so?

@dswij
Copy link
Member

dswij commented Aug 2, 2022

Well if the test is passing, I guess it's fine then. Thanks again for this!

@bors r+

@bors
Copy link
Contributor

bors commented Aug 2, 2022

📌 Commit 38e8013 has been approved by dswij

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Aug 2, 2022

⌛ Testing commit 38e8013 with merge 641e8a5...

bors added a commit that referenced this pull request Aug 2, 2022
…ws, r=dswij

Enable test empty_loop_no_std on windows

Verified that it actully works on windows
changelog: none
@bors
Copy link
Contributor

bors commented Aug 2, 2022

💔 Test failed - checks-action_test

@dswij
Copy link
Member

dswij commented Aug 2, 2022

@bors retry

@bors
Copy link
Contributor

bors commented Aug 2, 2022

⌛ Testing commit 38e8013 with merge 367d09f...

@bors
Copy link
Contributor

bors commented Aug 2, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: dswij
Pushing 367d09f to master...

@bors bors merged commit 367d09f into rust-lang:master Aug 2, 2022
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.

5 participants