Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jun 5, 2022

This renames the directories containing posix to unix; where applicable,
it also rename functions with the word "posix" to "unix"

Fixes #2012

@ghost ghost marked this pull request as draft June 5, 2022 20:34
This renames the directory containing posix to unix; where applicable,
it also rename functions with the word "posix" to "unix"
@RalfJung
Copy link
Member

RalfJung commented Jun 6, 2022

LGTM. :)

This PR is blocked until the merging of #2011

@landaire said they can deal with the conflicts. So I'd be fine landing this any time. I'm always happy to reduce the size of our PR list. ;)

@ghost ghost marked this pull request as ready for review June 6, 2022 08:49
@ghost
Copy link
Author

ghost commented Jun 6, 2022

Fixes #2012

@RalfJung
Copy link
Member

RalfJung commented Jun 6, 2022

Fixes #2012

This needs to go into the main PR comment; I have edited it accordingly.

@RalfJung
Copy link
Member

RalfJung commented Jun 6, 2022

Thanks for your first Miri PR. :-)

@bors r+

@ghost
Copy link
Author

ghost commented Jun 6, 2022

@bors ping

did it notice the "r+"?

@bors
Copy link
Contributor

bors commented Jun 6, 2022

😪 I'm awake I'm awake

@RalfJung
Copy link
Member

RalfJung commented Jun 6, 2022

Bors keeps missing r+'s recently...
@bors r+

@bors
Copy link
Contributor

bors commented Jun 6, 2022

📌 Commit d0a0369 has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Jun 6, 2022

⌛ Testing commit d0a0369 with merge fc8631a...

@bors
Copy link
Contributor

bors commented Jun 6, 2022

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing fc8631a to master...

@bors bors merged commit fc8631a into rust-lang:master Jun 6, 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.

Rename Posix → Unix

2 participants